eclair icon indicating copy to clipboard operation
eclair copied to clipboard

Published 20 hours ago • verified publisher icon ACINQ

[WIP] Handle mutual close in state `DATA_WAIT_FOR_FUNDING_CONFIRMED`

Open pm47 opened this issue 6 years ago • 3 comments

This is a clone of #358 which apparently lost track of the remote branch (unknown repository).

pm47 avatar Sep 25 '18 15:09 pm47

c537266 that's your favorite bug @btcontract

pm47 avatar Sep 25 '18 15:09 pm47

@btcontract I actually got bitten too! see 21c37e4

pm47 avatar Sep 26 '18 16:09 pm47

There are several more problems with this PR:

  • mutual close should also be supported in WAIT_FOR_FUNDING_LOCKED
  • the reconnection logic should handle the case where we have already sent a shutdown
  • I don't really like the fact that sending two CMD_CLOSE results in a force close; maybe we should rely on a feature bit to announce whether or not we support early close (cc @sstone)

pm47 avatar Sep 27 '18 11:09 pm47