search

lightningdevkit / ldk-node

A ready-to-go node implementation built using LDK.
Other
147 stars 73 forks source link

Consolidate LDK and BDK wallet syncs. #3

Closed tnull closed 1 year ago

tnull commented 2 years ago

Currently, the two wallet syncs are conducted independently in LdkLiteChainAccess.

As this may be partly redundant, we should try to consolidate the two wallet syncs. Doing this properly might depend on upstream changes (i.e. more traits in BDK), but we could at least try to do a 'dirty' version of this soon.

tnull commented 1 year ago

So, we should also look into making the existing blockchain traits (GetHeight, GetTx, etc.) Send, since this seems to be showstopper that hindered us from using them (vs. the 'raw' esplora client API methods) when migrating to the async BDK interface.

tnull commented 1 year ago

Closing, as partly addressed by better wallet integration, partly by the upcoming upstream lightning-transactions-sync crate.