chancloser: always set close output

We always need to set the close output in order for us to be able
to calculate the closing keys. Especially if there is only a dust
balance in BTC terms but perhaps a valuable amount in custom channel
funds.
This commit is contained in:
Oliver Gugger
2024-06-07 15:38:12 +02:00
parent 53dbd1ee66
commit d6e1d19fa1

View File

@ -432,9 +432,9 @@ func (c *ChanCloser) initChanShutdown() (*lnwire.Shutdown, error) {
return nil, err
}
// If the local balance isn't dust, then we'll track that we have an
// active close output.
if isDust, dustAmt := c.cfg.Channel.LocalBalanceDust(); !isDust {
// We'll track our local close output, even if it's dust in BTC terms,
// it might still carry value in custom channel terms.
_, dustAmt := c.cfg.Channel.LocalBalanceDust()
localBalance, _ := c.cfg.Channel.CommitBalances()
c.localCloseOutput = fn.Some(CloseOutput{
Amt: localBalance,
@ -442,7 +442,6 @@ func (c *ChanCloser) initChanShutdown() (*lnwire.Shutdown, error) {
PkScript: c.localDeliveryScript,
ShutdownRecords: shutdown.CustomRecords,
})
}
return shutdown, nil
}
@ -587,9 +586,9 @@ func (c *ChanCloser) ReceiveShutdown(msg lnwire.Shutdown) (
noShutdown := fn.None[lnwire.Shutdown]()
// If the remote balance isn't dust, then we'll track that they have an
// active close output.
if isDust, dustAmt := c.cfg.Channel.RemoteBalanceDust(); !isDust {
// We'll track their remote close output, even if it's dust in BTC
// terms, it might still carry value in custom channel terms.
_, dustAmt := c.cfg.Channel.RemoteBalanceDust()
_, remoteBalance := c.cfg.Channel.CommitBalances()
c.remoteCloseOutput = fn.Some(CloseOutput{
Amt: remoteBalance,
@ -597,7 +596,6 @@ func (c *ChanCloser) ReceiveShutdown(msg lnwire.Shutdown) (
PkScript: msg.Address,
ShutdownRecords: msg.CustomRecords,
})
}
switch c.state {
// If we're in the close idle state, and we're receiving a channel