diff --git a/config.go b/config.go index f206b43a5..6d6e93d7d 100644 --- a/config.go +++ b/config.go @@ -694,6 +694,7 @@ func DefaultConfig() Config { MaxChannelUpdateBurst: discovery.DefaultMaxChannelUpdateBurst, ChannelUpdateInterval: discovery.DefaultChannelUpdateInterval, SubBatchDelay: discovery.DefaultSubBatchDelay, + AnnouncementConf: discovery.DefaultProofMatureDelta, }, Invoices: &lncfg.Invoices{ HoldExpiryDelta: lncfg.DefaultHoldInvoiceExpiryDelta, @@ -1754,6 +1755,7 @@ func ValidateConfig(cfg Config, interceptor signal.Interceptor, fileParser, cfg.Invoices, cfg.Routing, cfg.Pprof, + cfg.Gossip, ) if err != nil { return nil, err diff --git a/discovery/gossiper.go b/discovery/gossiper.go index 9c5173439..d09dbee83 100644 --- a/discovery/gossiper.go +++ b/discovery/gossiper.go @@ -62,6 +62,11 @@ const ( // we'll maintain. This is the global size across all peers. We'll // allocate ~3 MB max to the cache. maxRejectedUpdates = 10_000 + + // DefaultProofMatureDelta specifies the default value used for + // ProofMatureDelta, which is the number of confirmations needed before + // processing the announcement signatures. + DefaultProofMatureDelta = 6 ) var ( @@ -1984,8 +1989,14 @@ func (d *AuthenticatedGossiper) addNode(msg *lnwire.NodeAnnouncement, // NOTE: must be used inside a lock. func (d *AuthenticatedGossiper) isPremature(chanID lnwire.ShortChannelID, delta uint32, msg *networkMsg) bool { - // TODO(roasbeef) make height delta 6 - // * or configurable + + // The channel is already confirmed at chanID.BlockHeight so we minus + // one block. For instance, if the required confirmation for this + // channel announcement is 6, we then only need to wait for 5 more + // blocks once the funding tx is confirmed. + if delta > 0 { + delta-- + } msgHeight := chanID.BlockHeight + delta diff --git a/lncfg/gossip.go b/lncfg/gossip.go index fbe5aae40..b8cab2c5f 100644 --- a/lncfg/gossip.go +++ b/lncfg/gossip.go @@ -1,12 +1,24 @@ package lncfg import ( + "fmt" "time" "github.com/lightningnetwork/lnd/discovery" "github.com/lightningnetwork/lnd/routing/route" ) +// minAnnouncementConf defines the minimal num of confs needed for the config +// AnnouncementConf. We choose 3 here as it's unlikely a reorg depth of 3 would +// happen. +// +// NOTE: The specs recommends setting this value to 6, which is the default +// value used for AnnouncementConf. However the receiver should be able to +// decide which channels to be included in its local graph, more details can be +// found: +// - https://github.com/lightning/bolts/pull/1215#issuecomment-2557337202 +const minAnnouncementConf = 3 + //nolint:ll type Gossip struct { PinnedSyncersRaw []string `long:"pinned-syncers" description:"A set of peers that should always remain in an active sync state, which can be used to closely synchronize the routing tables of two nodes. The value should be a hex-encoded pubkey, the flag can be specified multiple times to add multiple peers. Connected peers matching this pubkey will remain active for the duration of the connection and not count towards the NumActiveSyncer count."` @@ -18,6 +30,8 @@ type Gossip struct { ChannelUpdateInterval time.Duration `long:"channel-update-interval" description:"The interval used to determine how often lnd should allow a burst of new updates for a specific channel and direction."` SubBatchDelay time.Duration `long:"sub-batch-delay" description:"The duration to wait before sending the next announcement batch if there are multiple. Use a small value if there are a lot announcements and they need to be broadcast quickly."` + + AnnouncementConf uint32 `long:"announcement-conf" description:"The number of confirmations required before processing channel announcements."` } // Parse the pubkeys for the pinned syncers. @@ -35,3 +49,17 @@ func (g *Gossip) Parse() error { return nil } + +// Validate checks the Gossip configuration to ensure that the input values are +// sane. +func (g *Gossip) Validate() error { + if g.AnnouncementConf < minAnnouncementConf { + return fmt.Errorf("announcement-conf=%v must be no less than "+ + "%v", g.AnnouncementConf, minAnnouncementConf) + } + + return nil +} + +// Compile-time constraint to ensure Gossip implements the Validator interface. +var _ Validator = (*Gossip)(nil) diff --git a/sample-lnd.conf b/sample-lnd.conf index 86ea82485..7d431d87c 100644 --- a/sample-lnd.conf +++ b/sample-lnd.conf @@ -1729,6 +1729,8 @@ ; be broadcast quickly. ; gossip.sub-batch-delay=5s +; The number of confirmations required before processing channel announcements. +; gossip.announcement-conf=6 [invoices] diff --git a/server.go b/server.go index 9fd0b7a00..e79b25ac0 100644 --- a/server.go +++ b/server.go @@ -1118,7 +1118,7 @@ func newServer(cfg *Config, listenAddrs []net.Addr, return s.genNodeAnnouncement(nil) }, - ProofMatureDelta: 0, + ProofMatureDelta: cfg.Gossip.AnnouncementConf, TrickleDelay: time.Millisecond * time.Duration(cfg.TrickleDelay), RetransmitTicker: ticker.New(time.Minute * 30), RebroadcastInterval: time.Hour * 24,