From b75437dcb1cd7fc75787ed9add46d1dd339682fe Mon Sep 17 00:00:00 2001 From: Valentine Wallace Date: Thu, 4 Mar 2021 13:28:35 -0500 Subject: [PATCH] Raise max to_self_delay. lnd requires this to_self_delay for the max channel size (excluding wumbo). --- lightning/src/ln/channelmanager.rs | 9 +++++---- lightning/src/util/config.rs | 4 ++-- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/lightning/src/ln/channelmanager.rs b/lightning/src/ln/channelmanager.rs index de0ce5359..faae2c517 100644 --- a/lightning/src/ln/channelmanager.rs +++ b/lightning/src/ln/channelmanager.rs @@ -508,11 +508,12 @@ impl<'a> Drop for PersistenceNotifierGuard<'a> { } } -/// The amount of time we require our counterparty wait to claim their money (ie time between when -/// we, or our watchtower, must check for them having broadcast a theft transaction). +/// The amount of time in blocks we require our counterparty wait to claim their money (ie time +/// between when we, or our watchtower, must check for them having broadcast a theft transaction). pub(crate) const BREAKDOWN_TIMEOUT: u16 = 6 * 24; -/// The amount of time we're willing to wait to claim money back to us -pub(crate) const MAX_LOCAL_BREAKDOWN_TIMEOUT: u16 = 6 * 24 * 7; +/// The amount of time in blocks we're willing to wait to claim money back to us. This matches +/// the maximum required amount in lnd as of March 2021. +pub(crate) const MAX_LOCAL_BREAKDOWN_TIMEOUT: u16 = 2 * 6 * 24 * 7; /// The minimum number of blocks between an inbound HTLC's CLTV and the corresponding outbound /// HTLC's CLTV. This should always be a few blocks greater than channelmonitor::CLTV_CLAIM_BUFFER, diff --git a/lightning/src/util/config.rs b/lightning/src/util/config.rs index f43423d7c..19c456210 100644 --- a/lightning/src/util/config.rs +++ b/lightning/src/util/config.rs @@ -129,8 +129,8 @@ pub struct ChannelHandshakeLimits { /// Not checking this value would be a security issue, as our peer would be able to set it to /// max relative lock-time (a year) and we would "lose" money as it would be locked for a long time. /// - /// Default value: MAX_LOCAL_BREAKDOWN_TIMEOUT (1008), which we also enforce as a maximum value - /// so you can tweak config to reduce the loss of having useless locked funds (if your peer accepts) + /// Default value: 2016, which we also enforce as a maximum value so you can tweak config to + /// reduce the loss of having useless locked funds (if your peer accepts) pub their_to_self_delay: u16 } -- 2.39.5