Matt Corallo
Matt Corallo
> This is strange because before closing the channel, during the keysend, we wait for the payment to become successful (which happens during the PaymentSent and PaymentClaimed events), therefore we...
How many do y'all want? I'm of half a mind to just 10x it and call it a day.
> Given how this impacts resource usage, it makes sense for this to be a tunable parameter. I'm not entirely sure that it does impact resource usage that much. The...
> Well, would no limit at all be better? I mean, there is *some* cost, just not a hell of a lot. Not sure that actually no limit is the...
Okay, finally coming back to this post-0.2, sorry about that. I'm still not really convinced it makes sense to add a config knob here. IMO config knobs are not free...
Also happy to hop on a call and discuss this live rather than going back and forth endlessly in messages.
Hmm, I don't really think we can do this generically as a query - LDK doesn't keep around all the information that would be required to do this (and I...
Right, I was thinking we'd have a new event for splicing confirmations which could be used by clients, as you note.
Discussed with @tnull and we concluded its probably fine to just do a patch for ldk-node release and we can skip backporting this (which technically breaks compat).
There was some discussion on Discord, but we're not breaking backwards compatibility, only forwards compatibility. ie you can upgrade from 0.0.112 to the latest, but you can't upgrade then downgrade...