LN-Zap / zap-iOS

Zap Wallet - Native iOS lightning wallet focused on user experience and ease of use ⚡️
http://zaphq.io
MIT License
181 stars 47 forks source link

Wallet connected over Tor cannot read lightning invoices from clipboard or QR #343

Open initfortherekt opened 4 years ago

initfortherekt commented 4 years ago

Describe the bug When connected to a node over Tor, the send dialog cannot read lightning invoices from the clipboard or from a QR code. It can, however, read onchain addresses.

Connecting to the same node over clearnet and attempting an invoice spend resolves the issue.

To Reproduce Steps to reproduce the behavior:

  1. Connect wallet to a node over tor
  2. Click the Send button
  3. Attempt to read a lightning invoice from a QR code or from the clipboard
  4. Neither activity in step 3 will cause Zap to recognize a lightning invoice

Expected behavior Zap recognizes lightning invoices from QR code or from the clipboard when it is connected to a node over Tor

Your environment

initfortherekt commented 4 years ago

Update: LND logging indicates that in cases where this issue occurs, pathfinding has failed:

[ERR] RPCS: [/lnrpc.Lightning/QueryRoutes]: unable to find a path to destination

So I suppose the issue is that there is no feedback to the user that a route query is underway when they have scanned or pasted an invoice. I'm still not sure why this only occurs over Tor, when it appears to be an issue with the lnd host.