Skip to content

plugin: nwc: regularly rebroadcast info event#10689

Merged
ecdsa merged 2 commits into
spesmilo:masterfrom
f321x:rebroadcast_info_event
Jun 5, 2026
Merged

plugin: nwc: regularly rebroadcast info event#10689
ecdsa merged 2 commits into
spesmilo:masterfrom
f321x:rebroadcast_info_event

Conversation

@f321x
Copy link
Copy Markdown
Member

@f321x f321x commented Jun 5, 2026

I noticed with my long running NWC daemon that after a
couple days some clients relying on the info event being
available stopped working. Alby e.g. shows
"Error: no info event (kind 13194) returned from relay".
After restarting the daemon it works again for some days.
It seems that relays drop the info events after some days,
even though they are not ephemeral and don't have an expiry tag.
So rebroadcasting them once a day should make the NWC server more
reliable.

f321x added 2 commits June 5, 2026 12:20
It seemed weird to pass the taskgroup of the plugin into
the NWCServer, this looks a bit cleaner. It also allows
to get multiple tasks of the NWCServer cancelled by cancelling
the NWCServers taskgroup.
I noticed with my long running NWC deamon that after a
couple days some clients relying on the info event being
available stopped working. Alby e.g. shows
"Error: no info event (kind 13194) returned from relay".
After restarting the daemon it works again for some days.
It seems that relays drop the info events after some days,
even though they are not ephemeral and don't have an expiry tag.
So rebroadcasting them once a day should make the server more
reliable.
@f321x f321x force-pushed the rebroadcast_info_event branch from 9c839a4 to 60e3881 Compare June 5, 2026 10:20
@ecdsa
Copy link
Copy Markdown
Member

ecdsa commented Jun 5, 2026

lgtm

@ecdsa ecdsa merged commit 1ae56b8 into spesmilo:master Jun 5, 2026
11 checks passed
@f321x f321x deleted the rebroadcast_info_event branch June 5, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants