Skip to content

Use upstream Electrum EOF fix - #1055

Closed
tnull wants to merge 1 commit into
lightningdevkit:mainfrom
tnull:2026-08-electrum-client-eof-followup
Closed

tnull wants to merge 1 commit into
lightningdevkit:mainfrom
tnull:2026-08-electrum-client-eof-followup

Conversation

@tnull

@tnull tnull commented Aug 18, 2026 •

Copy link
Copy Markdown
Collaborator

Concurrent requests sharing an Electrum connection can remain blocked when the active reader sees a clean EOF. This can keep blocking Tokio workers alive through node shutdown.

Add a regression test based on the upstream fix.

Fixes #1007

Co-Authored-By: HAL 9000

Concurrent requests sharing an Electrum connection can remain blocked
when the active reader sees a clean EOF. This can keep blocking Tokio
workers alive through node shutdown.

Pin the upstream fix until it is released and cover the failure at the
LDK Node boundary so dependency updates cannot reintroduce the hang.

Fixes lightningdevkit#1007

Co-Authored-By: HAL 9000
@ldk-reviews-bot

ldk-reviews-bot commented Aug 18, 2026 •

Copy link
Copy Markdown

👋 Hi! This PR is now in draft status.
I'll wait to assign reviewers until you mark it as ready for review.
Just convert it out of draft status when you're ready for review!

@tnull

tnull commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator Author

Closing as superseded by #1182

@tnull tnull closed this Oct 20, 2026
tnull added a commit to tnull/ldk-node that referenced this pull request Oct 20, 2026
Concurrent requests sharing an Electrum connection can remain blocked
when the active reader sees a clean EOF. This can keep blocking Tokio
workers alive through node shutdown.

Require the released fix in electrum-client 0.25.1 and cover the failure
at the LDK Node boundary so dependency updates cannot reintroduce the
hang. BDK Electrum now supports this client version.

Fixes lightningdevkit#1007
Refs lightningdevkit#1055
(cherry picked from commit aff6ce0)
Co-Authored-By: HAL 9000
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.

Tokio runtime shutdown does not abort tasks created via spawn_blocking

2 participants