Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use all valid routes during blinded path construction #9334

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
+17 −32
Open

Use all valid routes during blinded path construction #9334

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
c0d0044
blindedpath: add the MaxNumPaths restriction on BuildBlindedPaymentPaths
MPins Dec 5, 2024
b7741af
invoicesrpc: included MaxNumPaths to be used on BuildBlindedPaymentPaths
MPins Dec 5, 2024
e6f8f8d
routing: no MaxNumPaths restriction on FindBlindedPaths
MPins Dec 5, 2024
415eddd
routing: modify the TestFindBlindedPathsWithMC to not use MaxNumPaths
MPins Dec 13, 2024
cfa17cb
lnd: added MaxNumPaths on blindedPathCfg to be used on AddInvoice
MPins Dec 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion routing/blindedpath/blinded_path.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,9 @@ type BuildBlindedPathCfg struct {
// route.
MinNumHops uint8

// MaxNumPaths is the maximum number of blinded paths to select.
MaxNumPaths uint8

// DefaultDummyHopPolicy holds the policy values that should be used for
// dummy hops in the cases where it cannot be derived via other means
// such as averaging the policy values of other hops on the path. This
Expand Down Expand Up @@ -132,8 +135,11 @@ func BuildBlindedPaymentPaths(cfg *BuildBlindedPathCfg) (
paths := make([]*zpay32.BlindedPaymentPath, 0, len(routes))

// For each route returned, we will construct the associated blinded
// payment path.
// payment path, until the maximum number of allowed paths.
MPins marked this conversation as resolved.
Show resolved Hide resolved
for _, route := range routes {
if len(paths) >= int(cfg.MaxNumPaths) {
break
}
// Extract the information we need from the route.
candidatePath := extractCandidatePath(route)

Expand Down