From 3b412ce50f94392f7fb2a0580d0c5db55dc65a88 Mon Sep 17 00:00:00 2001 From: Elle Mouton Date: Tue, 2 Mar 2021 14:11:10 +0200 Subject: [PATCH] multi: allow setting target confs for co-op close This commit adds a new config option: "--coop-close-target-confs" which allows a user to override the default target confirmations of 6 that is used to estimate a fee rate to use during a co-op closure initiated by a remote peer. --- config.go | 9 +++++++++ peer/brontide.go | 12 +++++++++--- sample-lnd.conf | 5 +++++ server.go | 1 + 4 files changed, 24 insertions(+), 3 deletions(-) diff --git a/config.go b/config.go index d3ab9b48..750d726d 100644 --- a/config.go +++ b/config.go @@ -89,6 +89,13 @@ const ( defaultAlias = "" defaultColor = "#3399FF" + // defaultCoopCloseTargetConfs is the default confirmation target + // that will be used to estimate a fee rate to use during a + // cooperative channel closure initiated by a remote peer. By default + // we'll set this to a lax value since we weren't the ones that + // initiated the channel closure. + defaultCoopCloseTargetConfs = 6 + // defaultHostSampleInterval is the default amount of time that the // HostAnnouncer will wait between DNS resolutions to check if the // backing IP of a host has changed. @@ -273,6 +280,7 @@ type Config struct { Color string `long:"color" description:"The color of the node in hex format (i.e. '#3399FF'). Used to customize node appearance in intelligence services"` MinChanSize int64 `long:"minchansize" description:"The smallest channel size (in satoshis) that we should accept. Incoming channels smaller than this will be rejected"` MaxChanSize int64 `long:"maxchansize" description:"The largest channel size (in satoshis) that we should accept. Incoming channels larger than this will be rejected"` + CoopCloseTargetConfs uint32 `long:"coop-close-target-confs" description:"The target number of blocks that a cooperative channel close transaction should confirm in. This is used to estimate the fee to use as the lower bound during fee negotiation for the channel closure."` DefaultRemoteMaxHtlcs uint16 `long:"default-remote-max-htlcs" description:"The default max_htlc applied when opening or accepting channels. This value limits the number of concurrent HTLCs that the remote party can add to the commitment. The maximum possible value is 483."` @@ -434,6 +442,7 @@ func DefaultConfig() Config { Color: defaultColor, MinChanSize: int64(funding.MinChanFundingSize), MaxChanSize: int64(0), + CoopCloseTargetConfs: defaultCoopCloseTargetConfs, DefaultRemoteMaxHtlcs: defaultRemoteMaxHtlcs, NumGraphSyncPeers: defaultMinPeers, HistoricalSyncInterval: discovery.DefaultHistoricalSyncInterval, diff --git a/peer/brontide.go b/peer/brontide.go index fda50811..12ec134f 100644 --- a/peer/brontide.go +++ b/peer/brontide.go @@ -298,6 +298,11 @@ type Config struct { // initiator for anchor channel commitments. MaxAnchorsCommitFeeRate chainfee.SatPerKWeight + // CoopCloseTargetConfs is the confirmation target that will be used + // to estimate the fee rate to use during a cooperative channel + // closure initiated by the remote peer. + CoopCloseTargetConfs uint32 + // ServerPubKey is the serialized, compressed public key of our lnd node. // It is used to determine which policy (channel edge) to pass to the // ChannelLink. @@ -2340,9 +2345,10 @@ func (p *Brontide) fetchActiveChanCloser(chanID lnwire.ChannelID) ( } // In order to begin fee negotiations, we'll first compute our - // target ideal fee-per-kw. We'll set this to a lax value, as - // we weren't the ones that initiated the channel closure. - feePerKw, err := p.cfg.FeeEstimator.EstimateFeePerKW(6) + // target ideal fee-per-kw. + feePerKw, err := p.cfg.FeeEstimator.EstimateFeePerKW( + p.cfg.CoopCloseTargetConfs, + ) if err != nil { peerLog.Errorf("unable to query fee estimator: %v", err) diff --git a/sample-lnd.conf b/sample-lnd.conf index 2925b5ee..9cddfcb5 100644 --- a/sample-lnd.conf +++ b/sample-lnd.conf @@ -252,6 +252,11 @@ ; to better align with your risk tolerance ; maxchansize= +; The target number of blocks in which a cooperative close initiated by a remote +; peer should be confirmed. This target is used to estimate the starting fee +; rate that will be used during fee negotiation with the peer. +; coop-close-target-confs=10 + ; The default max_htlc applied when opening or accepting channels. This value ; limits the number of concurrent HTLCs that the remote party can add to the ; commitment. The maximum possible value is 483. diff --git a/server.go b/server.go index ead9014a..7e138d03 100644 --- a/server.go +++ b/server.go @@ -3129,6 +3129,7 @@ func (s *server) peerConnected(conn net.Conn, connReq *connmgr.ConnReq, UnsafeReplay: s.cfg.UnsafeReplay, MaxOutgoingCltvExpiry: s.cfg.MaxOutgoingCltvExpiry, MaxChannelFeeAllocation: s.cfg.MaxChannelFeeAllocation, + CoopCloseTargetConfs: s.cfg.CoopCloseTargetConfs, MaxAnchorsCommitFeeRate: chainfee.SatPerKVByte( s.cfg.MaxCommitFeeRateAnchors * 1000).FeePerKWeight(), Quit: s.quit,