Remove direct bitcoind dependency, sync via electrum/esplora indexer#126
Open
bitwalt wants to merge 1 commit into
Open
Remove direct bitcoind dependency, sync via electrum/esplora indexer#126bitwalt wants to merge 1 commit into
bitwalt wants to merge 1 commit into
Conversation
Sync block/tx state via the configured indexer (electrum or esplora) through lightning-transaction-sync, dropping the lightning-block-sync RPC client and removing the bitcoind RPC fields from the unlock API, OpenAPI spec, README and Dockerfile. Async gossip verification is preserved on top of the new indexer client. Also stabilises two integration-test flakes that surface in long sequential runs: swap_roundtrip_multihop_asset_asset waits for all four channels to be usable before maker_init (both swap legs route over distinct RGB-asset channels), and upload_asset_media::fail tolerates the connection-reset that RequestBodyLimitLayer can produce mid-stream instead of a 413 response.
bitwalt
force-pushed
the
bitcoind-removal-indexer
branch
from
0e5ad71 to
9500239
Compare
zoedberg
requested changes
May 5, 2026
Member
zoedberg
left a comment
zoedberg
left a comment
There was a problem hiding this comment.
Please check my comment #125 (comment), I explained why we should not replace the block-sync with the transaction-sync but add it instead as an alternative sync mode. Please update this PR accordingly
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements #125.
Routes block + transaction sync through the configured indexer (electrum or esplora) via
lightning-transaction-sync, droppinglightning-block-sync(RPC mode) and removing the fourbitcoind_rpc_*fields from the unlock/init API, OpenAPI spec, README and Dockerfile. Async gossip verification, fee estimation, broadcast and UTXO lookup all go through the indexer client.