Matt Corallo
Matt Corallo
Reading `ChannelMonitor`s on startup is one of the slowest parts of LDK initialization. Now that we have an async `KVStore`, there's no need for that, we can simply paralellize their...
A handful of cleanups now that we bumped the MSRV and can use `impl Trait` in trait methods.
Event handlers in LDK are expected to always be idempotent, but LSPS2 service doesn't appear to be. eg once we get to `PaymentQueue::add_htlc` we `debug_assert` that an HTLC with the...
IIRC this is a separate crate for MSRV reasons, but I think we've bumped MSRV a few times since, so we can probably merge it into the main `lightning` crate...
dont forget to remove the docs saying async isnt supported once we get there https://github.com/lightningdevkit/rust-lightning/pull/2562#discussion_r1327098314
* What is `amt_to_forward_msat`, is that what I should use as `push_msat` (its not, but it really reads like it, in fact its not something the user has to deal...
@ariard pointed out in post-mere review that we could use more coverage at https://github.com/rust-bitcoin/rust-lightning/pull/762#pullrequestreview-595732739. Assigning @valentinewallace .
@devrandom noted at https://github.com/rust-bitcoin/rust-lightning/pull/931#issuecomment-851624586 that we currently cache the dns resolution result in `HttpClient` and never re-look-up the name after repeated connection failure. We should cache the `T: ToSocketAddrs` type...
We should have a mode where a `ChannelManager` will only take "safe" actions which can occur while the chain is being synchronized in the background (ie we are guaranteed that...
They're pretty confusing right now, see https://github.com/rust-bitcoin/rust-lightning/pull/314#discussion_r263630889