Move TODO from `handle_monitor_update_res` into `Channel`
authorMatt Corallo <git@bluematt.me>
Sat, 3 Dec 2022 05:38:24 +0000 (05:38 +0000)
committerMatt Corallo <git@bluematt.me>
Wed, 22 Feb 2023 00:51:13 +0000 (00:51 +0000)
commit46c6fb7f91b180fcae8358c82228008fa4eab2c4
tree0605422574cea08a32e7d567aa417a0da8dd0e13
parent9802afa53bac30e8bb831d267868b2aaabb61668
Move TODO from `handle_monitor_update_res` into `Channel`

The TODO mentioned in `handle_monitor_update_res` about how we
might forget about HTLCs in case of permanent monitor update
failure still applies in spite of all our changes. If a channel is
drop'd in general, monitor-pending updates may be lost if the
monitor update failed to persist.

This was always the case, and is ultimately the general form of the
the specific TODO, so we simply leave comments there
lightning/src/ln/channel.rs
lightning/src/ln/channelmanager.rs