Skip to content

Conversation

@ovitrif
Copy link

@ovitrif ovitrif commented Jan 26, 2026

Summary

  • Backport critical Electrum sync fix from upstream rust-lightning PR #4341
  • Bump version to 0.7.0-rc.18

Bug Fix

The ElectrumSyncClient was incorrectly treating mempool entries (height=0 or -1) from blockchain.scripthash.get_history as confirmed transactions. This caused get_merkle to fail since it cannot prove block inclusion for unconfirmed transactions, breaking sync for 0-conf channel funding transactions.

Changes

References

Test plan

  • cargo build succeeds
  • Bindings regenerated

Release

🤖 Generated with Claude Code

@ovitrif
Copy link
Author

ovitrif commented Jan 26, 2026

Release

@ovitrif ovitrif changed the title Backport upstream Electrum sync fix (PR #4341) fix: backport upstream electrum sync fix (PR #4341) Jan 26, 2026
@ovitrif ovitrif requested a review from ben-kaufman January 26, 2026 23:31
@ovitrif
Copy link
Author

ovitrif commented Jan 27, 2026

Going to merge after I open the android PR to integrate this and validate it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants