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.

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

[feature]: Phase out Legacy Payments #9136

Open
ziggie1984 opened this issue Sep 24, 2024 · 2 comments
Open

[feature]: Phase out Legacy Payments #9136

ziggie1984 opened this issue Sep 24, 2024 · 2 comments
Labels
enhancement Improvements to existing features / behaviour mpp

Comments

@ziggie1984
Copy link
Collaborator

Eclair and LDK stopped accepting legacy payments (payments without a payment_secret (payment_addr in lnd)). This can cause the LND mission_control system to fail the corresponding node because nodes not accepting Legacy Payment report the INVALID_ONION_PAYLOAD failure which causes our node to fail the complete node because we anticipate we did construct a right onion package.

case *lnwire.InvalidOnionPayload:
reportNode()

@ziggie1984 ziggie1984 added enhancement Improvements to existing features / behaviour mpp labels Sep 24, 2024
@ziggie1984
Copy link
Collaborator Author

Maybe instead of removing these payments make the MPP case default for all cases, and only with a special flag allow users to create legacy payments. Currently queryroutes or buildroute will not add a MPP record by default.

@ziggie1984
Copy link
Collaborator Author

Hmm actually this might also be a bug from our side looking at the node_announcment of eclair they clearly state:

  "14": {
                "name": "payment-addr",
                "is_required": true,
                "is_known": true
            }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvements to existing features / behaviour mpp
Projects
None yet
Development

No branches or pull requests

1 participant