16293817609

Committed 15 Jul 2025 12:53PM UTC coverage: 57.584% (-9.8%) from 67.338%

Build # 16293817609

Build Type

Pull #10031

github

Committed by

web-flow

Commit Message

Merge 8fd26da8b into df6c02e3a

Pull Request Pull Request #10031: Skip unnecessary disconnect and wait for disconnect to finish in shutdown

Run Details

36 of 40 new or added lines in 2 files covered. (90.0%)

28482 existing lines in 458 files now uncovered.

98681 of 171370 relevant lines covered (57.58%)

1.79 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

78.9

/routing/bandwidth.go

package routing

import (
        "fmt"

        "github.com/lightningnetwork/lnd/fn/v2"
        graphdb "github.com/lightningnetwork/lnd/graph/db"
        "github.com/lightningnetwork/lnd/htlcswitch"
        "github.com/lightningnetwork/lnd/lnwire"
        "github.com/lightningnetwork/lnd/routing/route"
        "github.com/lightningnetwork/lnd/tlv"
)

// bandwidthHints provides hints about the currently available balance in our
// channels.
type bandwidthHints interface {
        // availableChanBandwidth returns the total available bandwidth for a
        // channel and a bool indicating whether the channel hint was found.
        // The amount parameter is used to validate the outgoing htlc amount
        // that we wish to add to the channel against its flow restrictions. If
        // a zero amount is provided, the minimum htlc value for the channel
        // will be used. If the channel is unavailable, a zero amount is
        // returned.
        availableChanBandwidth(channelID uint64,
                amount lnwire.MilliSatoshi) (lnwire.MilliSatoshi, bool)

        // firstHopCustomBlob returns the custom blob for the first hop of the
        // payment, if available.
        firstHopCustomBlob() fn.Option[tlv.Blob]
}

// getLinkQuery is the function signature used to lookup a link.
type getLinkQuery func(lnwire.ShortChannelID) (
        htlcswitch.ChannelLink, error)

// bandwidthManager is an implementation of the bandwidthHints interface which
// uses the link lookup provided to query the link for our latest local channel
// balances.
type bandwidthManager struct {
        getLink       getLinkQuery
        localChans    map[lnwire.ShortChannelID]struct{}
        firstHopBlob  fn.Option[tlv.Blob]
        trafficShaper fn.Option[htlcswitch.AuxTrafficShaper]
}

// newBandwidthManager creates a bandwidth manager for the source node provided
// which is used to obtain hints from the lower layer w.r.t the available
// bandwidth of edges on the network. Currently, we'll only obtain bandwidth
// hints for the edges we directly have open ourselves. Obtaining these hints
// allows us to reduce the number of extraneous attempts as we can skip channels
// that are inactive, or just don't have enough bandwidth to carry the payment.
func newBandwidthManager(graph Graph, sourceNode route.Vertex,
        linkQuery getLinkQuery, firstHopBlob fn.Option[tlv.Blob],
        ts fn.Option[htlcswitch.AuxTrafficShaper]) (*bandwidthManager,
        error) {

        manager := &bandwidthManager{
                getLink:       linkQuery,
                localChans:    make(map[lnwire.ShortChannelID]struct{}),
                firstHopBlob:  firstHopBlob,
                trafficShaper: ts,
        }

        // First, we'll collect the set of outbound edges from the target
        // source node and add them to our bandwidth manager's map of channels.
        err := graph.ForEachNodeDirectedChannel(
                sourceNode, func(channel *graphdb.DirectedChannel) error {
                        shortID := lnwire.NewShortChanIDFromInt(
                                channel.ChannelID,
                        )
                        manager.localChans[shortID] = struct{}{}

                        return nil
                }, func() {
                        clear(manager.localChans)
                },
        )
        if err != nil {
                return nil, err
        }

        return manager, nil
}

// getBandwidth queries the current state of a link and gets its currently
// available bandwidth. Note that this function assumes that the channel being
// queried is one of our local channels, so any failure to retrieve the link
// is interpreted as the link being offline.
func (b *bandwidthManager) getBandwidth(cid lnwire.ShortChannelID,
        amount lnwire.MilliSatoshi) (lnwire.MilliSatoshi, error) {

        link, err := b.getLink(cid)
        if err != nil {
                return 0, fmt.Errorf("error querying switch for link: %w", err)
        }

        // If the link is found within the switch, but it isn't yet eligible
        // to forward any HTLCs, then we'll treat it as if it isn't online in
        // the first place.
        if !link.EligibleToForward() {
                return 0, fmt.Errorf("link not eligible to forward")
        }

        // bandwidthResult is an inline type that we'll use to pass the
        // bandwidth result from the external traffic shaper to the main logic
        // below.
        type bandwidthResult struct {
                // bandwidth is the available bandwidth for the channel as
                // reported by the external traffic shaper. If the external
                // traffic shaper is not handling the channel, this value will
                // be fn.None
                bandwidth fn.Option[lnwire.MilliSatoshi]

                // htlcAmount is the amount we're going to use to check if we
                // can add another HTLC to the channel. If the external traffic
                // shaper is handling the channel, we'll use 0 to just sanity
                // check the number of HTLCs on the channel, since we don't know
                // the actual HTLC amount that will be sent.
                htlcAmount fn.Option[lnwire.MilliSatoshi]
        }

        var (
                // We will pass the link bandwidth to the external traffic
                // shaper. This is the current best estimate for the available
                // bandwidth for the link.
                linkBandwidth = link.Bandwidth()

                bandwidthErr = func(err error) fn.Result[bandwidthResult] {
                        return fn.Err[bandwidthResult](err)
                }
        )

        result, err := fn.MapOptionZ(
                b.trafficShaper,
                func(s htlcswitch.AuxTrafficShaper) fn.Result[bandwidthResult] {
                        auxBandwidth, err := link.AuxBandwidth(
                                amount, cid, b.firstHopBlob, s,
                        ).Unpack()
                        if err != nil {
                                return bandwidthErr(fmt.Errorf("failed to get "+
                                        "auxiliary bandwidth: %w", err))
                        }

                        // If the external traffic shaper is not handling the
                        // channel, we'll just return the original bandwidth and
                        // no custom amount.
                        if !auxBandwidth.IsHandled {
                                return fn.Ok(bandwidthResult{})
                        }

                        // We don't know the actual HTLC amount that will be
                        // sent using the custom channel. But we'll still want
                        // to make sure we can add another HTLC, using the
                        // MayAddOutgoingHtlc method below. Passing 0 into that
                        // method will use the minimum HTLC value for the
                        // channel, which is okay to just check we don't exceed
                        // the max number of HTLCs on the channel. A proper
                        // balance check is done elsewhere.
                        return fn.Ok(bandwidthResult{
                                bandwidth:  auxBandwidth.Bandwidth,
                                htlcAmount: fn.Some[lnwire.MilliSatoshi](0),
                        })
                },
        ).Unpack()
        if err != nil {
                return 0, fmt.Errorf("failed to consult external traffic "+
                        "shaper: %w", err)
        }

        htlcAmount := result.htlcAmount.UnwrapOr(amount)

        // If our link isn't currently in a state where it can add another
        // outgoing htlc, treat the link as unusable.
        if err := link.MayAddOutgoingHtlc(htlcAmount); err != nil {
                return 0, fmt.Errorf("cannot add outgoing htlc to channel %v "+
                        "with amount %v: %w", cid, htlcAmount, err)
        }

        // If the external traffic shaper determined the bandwidth, we'll return
        // that value, even if it is zero (which would mean no bandwidth is
        // available on that channel).
        reportedBandwidth := result.bandwidth.UnwrapOr(linkBandwidth)

        return reportedBandwidth, nil
}

// availableChanBandwidth returns the total available bandwidth for a channel
// and a bool indicating whether the channel hint was found. If the channel is
// unavailable, a zero amount is returned.
func (b *bandwidthManager) availableChanBandwidth(channelID uint64,
        amount lnwire.MilliSatoshi) (lnwire.MilliSatoshi, bool) {

        shortID := lnwire.NewShortChanIDFromInt(channelID)
        _, ok := b.localChans[shortID]
        if !ok {
                return 0, false
        }

        bandwidth, err := b.getBandwidth(shortID, amount)
        if err != nil {
                log.Warnf("failed to get bandwidth for channel %v: %v",
                        shortID, err)

                return 0, true
        }

        return bandwidth, true
}

// firstHopCustomBlob returns the custom blob for the first hop of the payment,
// if available.
func (b *bandwidthManager) firstHopCustomBlob() fn.Option[tlv.Blob] {
        return b.firstHopBlob
}

1	package routing
2
3	import (
4	"fmt"
5
6	"github.com/lightningnetwork/lnd/fn/v2"
7	graphdb "github.com/lightningnetwork/lnd/graph/db"
8	"github.com/lightningnetwork/lnd/htlcswitch"
9	"github.com/lightningnetwork/lnd/lnwire"
10	"github.com/lightningnetwork/lnd/routing/route"
11	"github.com/lightningnetwork/lnd/tlv"
12	)
13
14	// bandwidthHints provides hints about the currently available balance in our
15	// channels.
16	type bandwidthHints interface {
17	// availableChanBandwidth returns the total available bandwidth for a
18	// channel and a bool indicating whether the channel hint was found.
19	// The amount parameter is used to validate the outgoing htlc amount
20	// that we wish to add to the channel against its flow restrictions. If
21	// a zero amount is provided, the minimum htlc value for the channel
22	// will be used. If the channel is unavailable, a zero amount is
23	// returned.
24	availableChanBandwidth(channelID uint64,
25	amount lnwire.MilliSatoshi) (lnwire.MilliSatoshi, bool)
26
27	// firstHopCustomBlob returns the custom blob for the first hop of the
28	// payment, if available.
29	firstHopCustomBlob() fn.Option[tlv.Blob]
30	}
31
32	// getLinkQuery is the function signature used to lookup a link.
33	type getLinkQuery func(lnwire.ShortChannelID) (
34	htlcswitch.ChannelLink, error)
35
36	// bandwidthManager is an implementation of the bandwidthHints interface which
37	// uses the link lookup provided to query the link for our latest local channel
38	// balances.
39	type bandwidthManager struct {
40	getLink getLinkQuery
41	localChans map[lnwire.ShortChannelID]struct{}
42	firstHopBlob fn.Option[tlv.Blob]
43	trafficShaper fn.Option[htlcswitch.AuxTrafficShaper]
44	}
45
46	// newBandwidthManager creates a bandwidth manager for the source node provided
47	// which is used to obtain hints from the lower layer w.r.t the available
48	// bandwidth of edges on the network. Currently, we'll only obtain bandwidth
49	// hints for the edges we directly have open ourselves. Obtaining these hints
50	// allows us to reduce the number of extraneous attempts as we can skip channels
51	// that are inactive, or just don't have enough bandwidth to carry the payment.
52	func newBandwidthManager(graph Graph, sourceNode route.Vertex,
53	linkQuery getLinkQuery, firstHopBlob fn.Option[tlv.Blob],
54	ts fn.Option[htlcswitch.AuxTrafficShaper]) (*bandwidthManager,
55	error) {	3✔
56		3✔
57	manager := &bandwidthManager{	3✔
58	getLink: linkQuery,	3✔
59	localChans: make(map[lnwire.ShortChannelID]struct{}),	3✔
60	firstHopBlob: firstHopBlob,	3✔
61	trafficShaper: ts,	3✔
62	}	3✔
63		3✔
64	// First, we'll collect the set of outbound edges from the target	3✔
65	// source node and add them to our bandwidth manager's map of channels.	3✔
66	err := graph.ForEachNodeDirectedChannel(	3✔
67	sourceNode, func(channel *graphdb.DirectedChannel) error {	6✔
68	shortID := lnwire.NewShortChanIDFromInt(	3✔
69	channel.ChannelID,	3✔
70	)	3✔
71	manager.localChans[shortID] = struct{}{}	3✔
72		3✔
73	return nil	3✔
74	}, func() {	6✔
75	clear(manager.localChans)	3✔
76	},	3✔
77	)
78	if err != nil {	3✔
79	return nil, err	×
80	}	×
81
82	return manager, nil	3✔
83	}
84
85	// getBandwidth queries the current state of a link and gets its currently
86	// available bandwidth. Note that this function assumes that the channel being
87	// queried is one of our local channels, so any failure to retrieve the link
88	// is interpreted as the link being offline.
89	func (b *bandwidthManager) getBandwidth(cid lnwire.ShortChannelID,
90	amount lnwire.MilliSatoshi) (lnwire.MilliSatoshi, error) {	3✔
91		3✔
92	link, err := b.getLink(cid)	3✔
93	if err != nil {	6✔
94	return 0, fmt.Errorf("error querying switch for link: %w", err)	3✔
95	}	3✔
96
97	// If the link is found within the switch, but it isn't yet eligible
98	// to forward any HTLCs, then we'll treat it as if it isn't online in
99	// the first place.
100	if !link.EligibleToForward() {	6✔
101	return 0, fmt.Errorf("link not eligible to forward")	3✔
102	}	3✔
103
104	// bandwidthResult is an inline type that we'll use to pass the
105	// bandwidth result from the external traffic shaper to the main logic
106	// below.
107	type bandwidthResult struct {	3✔
108	// bandwidth is the available bandwidth for the channel as	3✔
109	// reported by the external traffic shaper. If the external	3✔
110	// traffic shaper is not handling the channel, this value will	3✔
111	// be fn.None	3✔
112	bandwidth fn.Option[lnwire.MilliSatoshi]	3✔
113		3✔
114	// htlcAmount is the amount we're going to use to check if we	3✔
115	// can add another HTLC to the channel. If the external traffic	3✔
116	// shaper is handling the channel, we'll use 0 to just sanity	3✔
117	// check the number of HTLCs on the channel, since we don't know	3✔
118	// the actual HTLC amount that will be sent.	3✔
119	htlcAmount fn.Option[lnwire.MilliSatoshi]	3✔
120	}	3✔
121		3✔
122	var (	3✔
123	// We will pass the link bandwidth to the external traffic	3✔
124	// shaper. This is the current best estimate for the available	3✔
125	// bandwidth for the link.	3✔
126	linkBandwidth = link.Bandwidth()	3✔
127		3✔
128	bandwidthErr = func(err error) fn.Result[bandwidthResult] {	3✔
129	return fn.Err[bandwidthResult](err)	×
130	}	×
131	)
132
133	result, err := fn.MapOptionZ(	3✔
134	b.trafficShaper,	3✔
135	func(s htlcswitch.AuxTrafficShaper) fn.Result[bandwidthResult] {	3✔
UNCOV 136	auxBandwidth, err := link.AuxBandwidth(	×
UNCOV 137	amount, cid, b.firstHopBlob, s,	×
UNCOV 138	).Unpack()	×
UNCOV 139	if err != nil {	×
140	return bandwidthErr(fmt.Errorf("failed to get "+	×
141	"auxiliary bandwidth: %w", err))	×
142	}	×
143
144	// If the external traffic shaper is not handling the
145	// channel, we'll just return the original bandwidth and
146	// no custom amount.
UNCOV 147	if !auxBandwidth.IsHandled {	×
UNCOV 148	return fn.Ok(bandwidthResult{})	×
UNCOV 149	}	×
150
151	// We don't know the actual HTLC amount that will be
152	// sent using the custom channel. But we'll still want
153	// to make sure we can add another HTLC, using the
154	// MayAddOutgoingHtlc method below. Passing 0 into that
155	// method will use the minimum HTLC value for the
156	// channel, which is okay to just check we don't exceed
157	// the max number of HTLCs on the channel. A proper
158	// balance check is done elsewhere.
159	return fn.Ok(bandwidthResult{	×
160	bandwidth: auxBandwidth.Bandwidth,	×
161	htlcAmount: fn.Some[lnwire.MilliSatoshi](0),	×
162	})	×
163	},
164	).Unpack()
165	if err != nil {	3✔
166	return 0, fmt.Errorf("failed to consult external traffic "+	×
167	"shaper: %w", err)	×
168	}	×
169
170	htlcAmount := result.htlcAmount.UnwrapOr(amount)	3✔
171		3✔
172	// If our link isn't currently in a state where it can add another	3✔
173	// outgoing htlc, treat the link as unusable.	3✔
174	if err := link.MayAddOutgoingHtlc(htlcAmount); err != nil {	6✔
175	return 0, fmt.Errorf("cannot add outgoing htlc to channel %v "+	3✔
176	"with amount %v: %w", cid, htlcAmount, err)	3✔
177	}	3✔
178
179	// If the external traffic shaper determined the bandwidth, we'll return
180	// that value, even if it is zero (which would mean no bandwidth is
181	// available on that channel).
182	reportedBandwidth := result.bandwidth.UnwrapOr(linkBandwidth)	3✔
183		3✔
184	return reportedBandwidth, nil	3✔
185	}
186
187	// availableChanBandwidth returns the total available bandwidth for a channel
188	// and a bool indicating whether the channel hint was found. If the channel is
189	// unavailable, a zero amount is returned.
190	func (b *bandwidthManager) availableChanBandwidth(channelID uint64,
191	amount lnwire.MilliSatoshi) (lnwire.MilliSatoshi, bool) {	3✔
192		3✔
193	shortID := lnwire.NewShortChanIDFromInt(channelID)	3✔
194	_, ok := b.localChans[shortID]	3✔
195	if !ok {	3✔
UNCOV 196	return 0, false	×
UNCOV 197	}	×
198
199	bandwidth, err := b.getBandwidth(shortID, amount)	3✔
200	if err != nil {	6✔
201	log.Warnf("failed to get bandwidth for channel %v: %v",	3✔
202	shortID, err)	3✔
203		3✔
204	return 0, true	3✔
205	}	3✔
206
207	return bandwidth, true	3✔
208	}
209
210	// firstHopCustomBlob returns the custom blob for the first hop of the payment,
211	// if available.
212	func (b *bandwidthManager) firstHopCustomBlob() fn.Option[tlv.Blob] {	3✔
213	return b.firstHopBlob	3✔
214	}	3✔

lightningnetwork / lnd / 16293817609

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous