Skip to content

Consider maximum path length limit during path finding. #1371

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.

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

Closed
tnull opened this issue Mar 19, 2022 · 0 comments · Fixed by #1476
Closed

Consider maximum path length limit during path finding. #1371

tnull opened this issue Mar 19, 2022 · 0 comments · Fixed by #1476
Milestone

Comments

@tnull
Copy link
Contributor

tnull commented Mar 19, 2022

Currently, the maximum length of candidate paths is not considered during path finding, which could result in some paths being returned from get_route that are then immediately failing during construction of the onion. (Note: The only place the max. length is currently mentioned is this comment, which however refers to the pre-TLV maximum of 20.)

To solve this, we should try to approximate the TLV-onion length of candidate paths in get_route and pre-filter the results so that other viable paths are preferred.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants