16123414327

Committed 07 Jul 2025 05:12PM UTC coverage: 57.802% (+0.02%) from 57.787%

Build # 16123414327

Build Type

Pull #10042

github

Committed by

web-flow

Commit Message

Merge 5d5425994 into b815109b8

Pull Request Pull Request #10042: routing: add logs to debug potential payment sending issue

Run Details

21 of 37 new or added lines in 3 files covered. (56.76%)

56 existing lines in 11 files now uncovered.

98507 of 170420 relevant lines covered (57.8%)

1.79 hits per line

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

78.3

/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
                })

        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, 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("failed to get bandwidth for channel %v: "+
                        "link not eligible to forward", cid)
        }

        // 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 get bandwidth from external "+
                        "traffic shaper for channel %v: %w", cid, 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 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(sourceNode,	3✔
67	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	})	3✔
75
76	if err != nil {	3✔
77	return nil, err	×
78	}	×
79
80	return manager, nil	3✔
81	}
82
83	// getBandwidth queries the current state of a link and gets its currently
84	// available bandwidth. Note that this function assumes that the channel being
85	// queried is one of our local channels, so any failure to retrieve the link
86	// is interpreted as the link being offline.
87	func (b *bandwidthManager) getBandwidth(cid lnwire.ShortChannelID,
88	amount lnwire.MilliSatoshi) (lnwire.MilliSatoshi, error) {	3✔
89		3✔
90	link, err := b.getLink(cid)	3✔
91	if err != nil {	6✔
92	return 0, err	3✔
93	}	3✔
94
95	// If the link is found within the switch, but it isn't yet eligible
96	// to forward any HTLCs, then we'll treat it as if it isn't online in
97	// the first place.
98	if !link.EligibleToForward() {	6✔
99	return 0, fmt.Errorf("failed to get bandwidth for channel %v: "+	3✔
100	"link not eligible to forward", cid)	3✔
101	}	3✔
102
103	// bandwidthResult is an inline type that we'll use to pass the
104	// bandwidth result from the external traffic shaper to the main logic
105	// below.
106	type bandwidthResult struct {	3✔
107	// bandwidth is the available bandwidth for the channel as	3✔
108	// reported by the external traffic shaper. If the external	3✔
109	// traffic shaper is not handling the channel, this value will	3✔
110	// be fn.None	3✔
111	bandwidth fn.Option[lnwire.MilliSatoshi]	3✔
112		3✔
113	// htlcAmount is the amount we're going to use to check if we	3✔
114	// can add another HTLC to the channel. If the external traffic	3✔
115	// shaper is handling the channel, we'll use 0 to just sanity	3✔
116	// check the number of HTLCs on the channel, since we don't know	3✔
117	// the actual HTLC amount that will be sent.	3✔
118	htlcAmount fn.Option[lnwire.MilliSatoshi]	3✔
119	}	3✔
120		3✔
121	var (	3✔
122	// We will pass the link bandwidth to the external traffic	3✔
123	// shaper. This is the current best estimate for the available	3✔
124	// bandwidth for the link.	3✔
125	linkBandwidth = link.Bandwidth()	3✔
126		3✔
127	bandwidthErr = func(err error) fn.Result[bandwidthResult] {	3✔
128	return fn.Err[bandwidthResult](err)	×
129	}	×
130	)
131
132	result, err := fn.MapOptionZ(	3✔
133	b.trafficShaper,	3✔
134	func(s htlcswitch.AuxTrafficShaper) fn.Result[bandwidthResult] {	3✔
135	auxBandwidth, err := link.AuxBandwidth(	×
136	amount, cid, b.firstHopBlob, s,	×
137	).Unpack()	×
138	if err != nil {	×
139	return bandwidthErr(fmt.Errorf("failed to get "+	×
140	"auxiliary bandwidth: %w", err))	×
141	}	×
142
143	// If the external traffic shaper is not handling the
144	// channel, we'll just return the original bandwidth and
145	// no custom amount.
146	if !auxBandwidth.IsHandled {	×
147	return fn.Ok(bandwidthResult{})	×
148	}	×
149
150	// We don't know the actual HTLC amount that will be
151	// sent using the custom channel. But we'll still want
152	// to make sure we can add another HTLC, using the
153	// MayAddOutgoingHtlc method below. Passing 0 into that
154	// method will use the minimum HTLC value for the
155	// channel, which is okay to just check we don't exceed
156	// the max number of HTLCs on the channel. A proper
157	// balance check is done elsewhere.
158	return fn.Ok(bandwidthResult{	×
159	bandwidth: auxBandwidth.Bandwidth,	×
160	htlcAmount: fn.Some[lnwire.MilliSatoshi](0),	×
161	})	×
162	},
163	).Unpack()
164	if err != nil {	3✔
NEW 165	return 0, fmt.Errorf("failed to get bandwidth from external "+	×
NEW 166	"traffic shaper for channel %v: %w", cid, err)	×
UNCOV 167	}	×
168
169	htlcAmount := result.htlcAmount.UnwrapOr(amount)	3✔
170		3✔
171	// If our link isn't currently in a state where it can add another	3✔
172	// outgoing htlc, treat the link as unusable.	3✔
173	if err := link.MayAddOutgoingHtlc(htlcAmount); err != nil {	6✔
174	return 0, fmt.Errorf("cannot add outgoing htlc to channel %v "+	3✔
175	"with amount %v: %w", cid, htlcAmount, err)	3✔
176	}	3✔
177
178	// If the external traffic shaper determined the bandwidth, we'll return
179	// that value, even if it is zero (which would mean no bandwidth is
180	// available on that channel).
181	reportedBandwidth := result.bandwidth.UnwrapOr(linkBandwidth)	3✔
182		3✔
183	return reportedBandwidth, nil	3✔
184	}
185
186	// availableChanBandwidth returns the total available bandwidth for a channel
187	// and a bool indicating whether the channel hint was found. If the channel is
188	// unavailable, a zero amount is returned.
189	func (b *bandwidthManager) availableChanBandwidth(channelID uint64,
190	amount lnwire.MilliSatoshi) (lnwire.MilliSatoshi, bool) {	3✔
191		3✔
192	shortID := lnwire.NewShortChanIDFromInt(channelID)	3✔
193	_, ok := b.localChans[shortID]	3✔
194	if !ok {	3✔
195	return 0, false	×
196	}	×
197
198	bandwidth, err := b.getBandwidth(shortID, amount)	3✔
199	if err != nil {	6✔
200	log.Warnf("failed to get bandwidth for channel %v: %v",	3✔
201	shortID, err)	3✔
202	}	3✔
203
204	return bandwidth, true	3✔
205	}
206
207	// firstHopCustomBlob returns the custom blob for the first hop of the payment,
208	// if available.
209	func (b *bandwidthManager) firstHopCustomBlob() fn.Option[tlv.Blob] {	3✔
210	return b.firstHopBlob	3✔
211	}	3✔

lightningnetwork / lnd / 16123414327

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