Raise max to_self_delay.
authorValentine Wallace <vwallace@protonmail.com>
Thu, 4 Mar 2021 18:28:35 +0000 (13:28 -0500)
committerValentine Wallace <vwallace@protonmail.com>
Mon, 8 Mar 2021 22:30:02 +0000 (17:30 -0500)
lnd requires this to_self_delay for the max channel size
(excluding wumbo).

lightning/src/ln/channelmanager.rs
lightning/src/util/config.rs

index de0ce53590dcebfad11af1ca78e11c93547a4ad2..faae2c51726fb03dd0e3a3d93cfd0209fbb11925 100644 (file)
@@ -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,
index f43423d7c0c0af923a4ac8f08a562aea474f946c..19c4562105bff3aa15b1d6f88099a8da42a33669 100644 (file)
@@ -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
 }