diff --git a/packages/nostr/NOTES.md b/packages/nostr/NOTES.md deleted file mode 100644 index 7db31fa1..00000000 --- a/packages/nostr/NOTES.md +++ /dev/null @@ -1,2 +0,0 @@ -- Don't use arrays for callbacks because there's no way to clear them, -passing undefined or null to on* should clear the callback diff --git a/packages/nostr/src/client/conn.ts b/packages/nostr/src/client/conn.ts index 75ff313c..4e3f3309 100644 --- a/packages/nostr/src/client/conn.ts +++ b/packages/nostr/src/client/conn.ts @@ -49,7 +49,9 @@ export class Conn { if (err instanceof ProtocolError) { this.#errorCallback?.(err) } else { - // TODO Not sure if this is the case? + // TODO Not sure if this is the best idea. + // Investigate what WebSocket does if the callback throws? + // Either way it seems like the best idea is to have `onError` called on all types of errors throw err } } diff --git a/packages/nostr/src/client/index.ts b/packages/nostr/src/client/index.ts index 2fd7058e..b07fa167 100644 --- a/packages/nostr/src/client/index.ts +++ b/packages/nostr/src/client/index.ts @@ -226,7 +226,7 @@ interface ConnState { */ auth: boolean /** - * Should this connection be used for receiving messages? + * Should this connection be used for receiving events? */ read: boolean /**