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

bolt12: Zeus sends sats when user specifies fiat #2762

Closed
chrisguida opened this issue Jan 22, 2025 · 0 comments · Fixed by #2765
Closed

bolt12: Zeus sends sats when user specifies fiat #2762

chrisguida opened this issue Jan 22, 2025 · 0 comments · Fixed by #2765
Labels
Bug Something isn't working

Comments

@chrisguida
Copy link

Describe the bug

When paying bolt12 offers, the user has the option of specifying the amount they'd like to pay in their fiat currency of choice, or sats, or BTC.

When specifying in fiat, Zeus appears to do the conversion into sats properly, but then when the invoice is actually fetched, the invoice is denominated in sats, with the amount the user entered for fiat. So, for example, if a user is attempting to send US$10 to the bolt12 receiver, the user selects dollars as their denomination and enters "10". Zeus appears to convert this properly to 9,577 sats using the USD exchange rate:

Image

However, when the user taps "proceed", the actual invoice fetched says 10 sats, rather than 9,577 sats, which has resulted in several hilariously small payments to my bolt12 offer:

Image

Reproduce

Follow steps above

ZEUS version

v0.9.4

Node interface

Core Lightning (CLNRest)

Network

Clearnet

Device

Android Pixel 8a

Device operating system

Android 15

Log output

@chrisguida chrisguida added the Bug Something isn't working label Jan 22, 2025
@kaloudis kaloudis linked a pull request Jan 23, 2025 that will close this issue
30 tasks
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

Successfully merging a pull request may close this issue.

1 participant