Skip to content
This repository has been archived by the owner on Apr 28, 2023. It is now read-only.

fix: trim "lightning:" prefix of invoices #171

Merged
merged 1 commit into from
Apr 22, 2019
Merged

Conversation

michael1011
Copy link
Member

@michael1011 michael1011 commented Apr 20, 2019

Closes BoltzExchange/boltz-backend#102

The lightning: prefix will be valid in the input field for the invoice and removed before making the request to the middleware.

@armurbalda
Copy link

Maybe some quick research on it's status to become a standard? If so we should treat it differently. Couldn't find anything in a quick search tho.

@michael1011
Copy link
Member Author

michael1011 commented Apr 20, 2019

Maybe some quick research on it's status to become a standard?

@armurbalda it's just a prefix for the invoice so that, if it's a link, the lightning wallets know that the invoice should be opened by them. If one remove removes the lightning: prefix, one gets the raw invoice that contains all of the needed information

Copy link
Contributor

@ImmanuelSegol ImmanuelSegol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@michael1011 michael1011 force-pushed the trim-lightning-prefix branch from 80de1b8 to cc6b360 Compare April 21, 2019 07:13
@michael1011 michael1011 merged commit da3e420 into master Apr 22, 2019
@michael1011 michael1011 deleted the trim-lightning-prefix branch April 22, 2019 08:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Lightning URI's starting with lightning:
4 participants