-
Notifications
You must be signed in to change notification settings - Fork 913
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
Using fetchinvoice on a Phoenix Bolt12 offer crashes CLN #7513
Comments
@hMsats bolt 12 should require a fetchOffer i suppose. |
While I did not get what @turizspace is trying to suggest, I a not able to find the commit where you are running your core lightning node. @hMsats can you add the |
Ok I was able to reproduce in testnet with bf54913
|
@vincenzopalazzo I think you're referring to the @turizspace CLN doesn't have |
I boldly commented out the following test in fetchinvoice.c (lines 229-232)
and now I first got:
without crashing CLN. Then I tried again and now it worked well:
Then first decoded the invoice:
I then tried to pay the invoice:
which at this stage is probably normal. So the only problem seems to be the code starting at line 229 in fetchinvoice.c |
Note: when I turn off Phoenix wallet on my phone, I correctly get:
|
Note: I'm able to pay "normal" invoices from CLN to my Phoenix wallet:
|
Possible solution: shouldn't |
…uer_id` Now we actually check the other fields too, as per BOLT! Reported-by: https://github.com/hMsats Fixes: ElementsProject#7513 Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Great report! We don't create an offer without an offer_issuer_id, so this (crap) code was untested :( Thanks for catching it before release!! |
…uer_id` Now we actually check the other fields too, as per BOLT! Reported-by: https://github.com/hMsats Fixes: ElementsProject#7513 Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
…uer_id` Now we actually check the other fields too, as per BOLT! Reported-by: https://github.com/hMsats Fixes: ElementsProject#7513 Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
I don't know if this is useful but my Phoenix log shows the following when I try to pay an invoice obtained via
all the time I see |
I know it's very early but I'm trying to help :-)
I'm trying to pay a bolt12 offer from the latest Phoenix wallet (from CLN to Phoenix)
Using master, for the first time
decode
on a Phoenix Bolt12 offer works well:but
fetchinvoice
crashes core lightning:Output:
getinfo
outputv24.05-213-gbf54913-modded
The text was updated successfully, but these errors were encountered: