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

Bug: LNDK doesn't know how to respond to an offer that lists itself as the intro node #123

Open
orbitalturtle opened this issue Jun 18, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@orbitalturtle
Copy link
Collaborator

orbitalturtle commented Jun 18, 2024

Describe the bug

Currently, if LND/LNDK is chosen as the introduction point in an offer it's trying to pay, it will throw an error. This is a bit of an edge case which is mostly likely to occur in a scenario where we're paying an offer from a node that we're directly connected with.

In this scenario, this is the error:

Error paying for offer: Status { code: Internal, message: "Internal error: Error connecting to peer: Status { code: Unknown, message: "cannot make connection to self", metadata: MetadataMap { headers: {"content-type": "application/grpc"} }, source: None }", metadata: MetadataMap { headers: {"content-type": "application/grpc", "date": "Tue, 18 Jun 2024 21:30:40 GMT", "content-length": "0"} }, source: None }

@orbitalturtle orbitalturtle added the bug Something isn't working label Jun 18, 2024
@jbesraa
Copy link
Contributor

jbesraa commented Jun 19, 2024

Maybe related to lightningdevkit/rust-lightning#2952

@tnull
Copy link

tnull commented Jun 19, 2024

Maybe related to lightningdevkit/rust-lightning#2952

Hm, no, actually not as the issue also occurs for announced channels, i.e., also in the case that works perfectly fine in LDK Node integration tests.

@orbitalturtle
Copy link
Collaborator Author

orbitalturtle commented Sep 9, 2024

Note that we should be able to fix this once we've upgraded to LDK v0.0.124 because advance_path_by_one for messages and payments are now exposed.

@carlaKC
Copy link
Collaborator

carlaKC commented Oct 7, 2024

I think you'll still run into issues with the LND payment itself?
Needs LND change: lightningnetwork/lnd#9027

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants