Drop non-anchor channel fee upper bound limit entirely 2023-10-no-chan-feerate-upper-bound
authorMatt Corallo <git@bluematt.me>
Tue, 31 Oct 2023 18:39:04 +0000 (18:39 +0000)
committerMatt Corallo <git@bluematt.me>
Sat, 11 Nov 2023 17:32:31 +0000 (17:32 +0000)
commitd30d599a2f5cfc16568e026d0764396a200d8388
treef2b87e5f4e6a48b992a9f3df6df494a1fd4ed180
parentb6f3d0a5fa6cf6036d317d3ff47e5252be47bc40
Drop non-anchor channel fee upper bound limit entirely

Quite a while ago we added checks for the total current dust
exposure on a channel to explicitly limit dust inflation attacks.
When we did this, we kept the existing upper bound on the channel's
feerate in place. However, these two things are redundant - the
point of the feerate upper bound is to prevent dust inflation, and
it does so in a crude way that can cause spurious force-closures.

Here we simply drop the upper bound entirely, relying on the dust
inflation limit to prevent dust inflation instead.
fuzz/src/chanmon_consistency.rs
fuzz/src/full_stack.rs
lightning/src/chain/chaininterface.rs
lightning/src/ln/channel.rs
lightning/src/util/test_utils.rs