aiolabs/lightning-pub
2
0
Fork 0
Code Issues 9 Pull requests 4 Projects Releases Packages Wiki Activity Actions

fix(nostr): close SimplePool after publishing to prevent connection leak

Browse source 
Each sendEvent() call created a new SimplePool() but never closed it,
causing relay WebSocket connections to accumulate indefinitely (~20/min).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Patrick Mulligan 2026-02-28 07:53:09 -05:00
parent 6512e10f08
commit 611eb4fc04
1 changed files with 17 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/services/nostr/nostrPool.ts
View file

@ -205,6 +205,7 @@ export class NostrPool {
const log = getLogger({ appName: keys.name }) const log = getLogger({ appName: keys.name })
this.log(`📤 Publishing Kind ${event.kind} event to ${relays.length} relay(s): ${relays.join(', ')}`) this.log(`📤 Publishing Kind ${event.kind} event to ${relays.length} relay(s): ${relays.join(', ')}`)
const pool = new SimplePool() const pool = new SimplePool()
try {
await Promise.all(pool.publish(relays, signed).map(async p => { await Promise.all(pool.publish(relays, signed).map(async p => {
try { try {
await p await p
@ -220,6 +221,9 @@ export class NostrPool {
} else { } else {
this.log(`✅ Kind ${event.kind} event published successfully (id: ${signed.id.slice(0, 16)}...)`) this.log(`✅ Kind ${event.kind} event published successfully (id: ${signed.id.slice(0, 16)}...)`)
} }
} finally {
pool.close(relays)
}
} }
private getRelays(initiator: SendInitiator, requestRelays?: string[]) { private getRelays(initiator: SendInitiator, requestRelays?: string[]) {