search

ZeusLN / zeus

A mobile Bitcoin wallet fit for the gods. ⚡️ Est. 563345
https://ZeusLN.com
Other
1.02k stars 145 forks source link

LNURLp & LNURLw causing app to crash #578

Closed thecharacter closed 2 years ago

thecharacter commented 3 years ago

Love the app ya'll I found whenever I scan a LNURLp or a LNURLw QR code generated from LNbits.com the app returns an error and causes the app to crash. I tested the same QR codes against Bluewallet and they worked.

I'm using a Iphone running 0.12.1-beta LND, Zeus version v0.6.0 alpha2

IMG_E6A9731DE9BB-1

pseudozach commented 3 years ago

I'm unable to reproduce this with any lnurlp/lnurlw string from lnbits. Even tried with the URL you mentioned here. Zeus is not crashing. Maybe it was an intermittent issue and lnbits returned something strange temporarily. Please share if you still encounter this.

thecharacter commented 3 years ago

Thanks for checking in on this issue. Here is the LNURL in question that causing Zeus to return an error. When scanning the LNURL the error message above loads ( it generates at least 34 times) multiple times. If I click OK on the error message 34 times the screen eventually appears where you can actually pay the invoice. Here's an unlisted video on youtube of what's actually happening. The LNURLp is below.

https://youtu.be/wGhCsLbjLQs

Screen Shot 2021-09-28 at 1 22 51 PM

pseudozach commented 3 years ago

This QR code reads OK on Android 11, Zeus v0.5.2 but I was able to see the same issue on Alpha2. Will look into this. Thanks for the detailed report!

kaloudis commented 2 years ago

Fixed in v0.6.0-alpha4 https://github.com/ZeusLN/zeus/releases/tag/v0.6.0-alpha4