ACINQ / eclair

A scala implementation of the Lightning Network.
Apache License 2.0
1.24k stars 266 forks source link

Add closing test reconnect with 3rd-stage txs #2820

Closed t-bast closed 8 months ago

t-bast commented 8 months ago

We had some cases where channels were staying stuck in the CLOSING state without going to the CLOSED state in previous versions of eclair where we published different versions of our 3rd-stage transactions after a restart and a feerate increase.

This happened in two cases:

This was on my backlog for a while, and I wanted to take another look at it after the splicing implementation. Fortunately, this was fixed as part of the splicing changes, but it's worth having a test to ensure that there's no regression from future changes.