Skip to content

Commit 722c693

Browse files
committed
Slightly increase the allowed fee upper limit on remote update_fees
At least lnd is fairly aggressively high in their (non-anchor) fee estimation, running the risk of force-closure. Further, because we rely on a fee estimator we don't have full control over, we should not assume our users' fees are particularly conservative, and thus give our peers a lot of leeway.
1 parent 0518001 commit 722c693

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lightning/src/ln/channel.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -669,7 +669,7 @@ impl<Signer: Sign> Channel<Signer> {
669669
if feerate_per_kw < lower_limit {
670670
return Err(ChannelError::Close(format!("Peer's feerate much too low. Actual: {}. Our expected lower limit: {}", feerate_per_kw, lower_limit)));
671671
}
672-
let upper_limit = fee_estimator.get_est_sat_per_1000_weight(ConfirmationTarget::HighPriority) as u64 * 2;
672+
let upper_limit = fee_estimator.get_est_sat_per_1000_weight(ConfirmationTarget::HighPriority) as u64 * 4;
673673
if feerate_per_kw as u64 > upper_limit {
674674
return Err(ChannelError::Close(format!("Peer's feerate much too high. Actual: {}. Our expected upper limit: {}", feerate_per_kw, upper_limit)));
675675
}

0 commit comments

Comments
 (0)