Allow sending closing tx signatures asynchronously
Another async signing PR, this time for sign_closing_transaction!
If you've reviewed the other PRs, you know the drill: try to go about business as usual, if we fail to get the signature, do everything except for send the message, and mark our signer_pending_closing flag. The user then needs to call signer_unblocked, and we'll try again to get the signature, then create + send the message. In this case we not only need to send the message, but also 1. broadcast the fully signed closing tx, 2. broadcast a channel update, and 3. remove the channel from our map.
I added some additional state to track everything we need to do everything we need (rebuild the tx, fully sign, send our closing_signed msg, etc.) upon the signer being unblocked. This state is not persisted.
Codecov Report
Attention: Patch coverage is 86.36364% with 9 lines in your changes missing coverage. Please review.
Project coverage is 91.12%. Comparing base (
33e6995) to head (6e2071a). Report is 108 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #3153 +/- ##
==========================================
+ Coverage 89.78% 91.12% +1.33%
==========================================
Files 123 126 +3
Lines 102330 114900 +12570
Branches 102330 114900 +12570
==========================================
+ Hits 91876 104701 +12825
+ Misses 7763 7708 -55
+ Partials 2691 2491 -200
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
rebased because of conflicts
Taking a look!
rebased because of conflicts as well
Feel free to squash, lets get this over the line!
addressed latest nits and squashed
Sorry for the delay, release process is always fun. I think this LGTM.
squashed