breez / breezmobile

Lightning Network mobile client
https://breez.technology
GNU General Public License v3.0
570 stars 131 forks source link

Increase timeout for LNURL withdraw #848

Closed theDavidCoen closed 2 years ago

theDavidCoen commented 2 years ago

When I scan a LNURL withdraw link I made with lnbits (on legend.lnbits.com), I get this warning. I'm able to scan it with other wallets, for example Wallet of Satoshi or Zeus. CsY0ibv2

I suggest to increase the timeout, but the fix should be general, not specific for that demo service.

You can use this LNURL withdraw link for testing: lightning:LNURL1DP68GURN8GHJ7MR9VAJKUEPWD3HXY6T5WVHXXMMD9AMKJARGV3EXZAE0V9CXJTMKXYHKCMN4WFKZ7M6YWQEN2EJHXEEXYKR60F38JETSGSM9JENY9UE55526DUE8W5R6V4SHY3ME2DZYVKNF2FR42Y8VK7R

Or create a new one on https://legend.lnbits.com (you need to fund a wallet and activate LNURL-withdraw plugin).

kingonly commented 2 years ago

@erdemyerebasmaz please look into it. I tested the link in WoS and it takes a while to load, cso our timeout might be too short for slow servers.

erdemyerebasmaz commented 2 years ago

Average response time for the query at hand proved to be ~20 seconds with various dns & proxy settings, low bandwidth tests. The timeout has been increased to 60 seconds. Screenshot_20220403_210543_com breez client debug

kingonly commented 2 years ago

Fixed in latest