Avoid writing `ChannelManager` when hitting lnd bug 6039
authorMatt Corallo <git@bluematt.me>
Thu, 14 Mar 2024 19:47:07 +0000 (19:47 +0000)
committerMatt Corallo <git@bluematt.me>
Tue, 9 Apr 2024 15:01:41 +0000 (15:01 +0000)
commit271103d66017e4622d42edc3aa8a3b850b104e3e
tree8f9a748081ec6993854585f0e6a838bf89bc724a
parent72c4eb83d82cb26d2c16023febb5373b1ae27caf
Avoid writing `ChannelManager` when hitting lnd bug 6039

When we hit lnd bug 6039, we end up sending error messages to peers
in a loop. This should be fine, but because we used the generic
`PersistenceNotifierGuard::notify_on_drop` lock above the specific
handling, we end up writing `ChannelManager` every time we manage a
round-trip to our peer.

This can add up quite quickly, and isn't actually changing, so we
really need to avoid writing the `ChannelManager` in this case.
lightning/src/ln/channelmanager.rs