Fonta1n3 / FullyNoded

Self sovereign, secure, powerful, easy to use wallet that utilizes your own node as a backend. Powered by PSBT's and descriptors. Acts as an offline signer using your node as a watch-only wallet. C-Lightning compatible for instant, unfairly cheap payments.
https://fullynoded.app
Other
197 stars 37 forks source link

SSL error when connecting to LND via Tor #188

Closed aetikon closed 2 years ago

aetikon commented 2 years ago

Describe the bug

We get an SSL error when attempting to connect to LND on MyNodeBTC.

To Reproduce Steps to reproduce the behavior:

  1. In myNodeBtc, pair a wallet using Lightning (REST + Tor)
  2. Load the QR code in FN
  3. See error

Expected behavior

No error

Screenshots

FN-error

Device:

Additional context

Apparently the alert is triggered in View Controllers/Settings/Node Manager/Lightning/AddPeerViewController.swift:174

Fonta1n3 commented 2 years ago

Ive also experienced this issue on a rare occasion. For me rebooting Tor on the server seemed to always fix the issue. Can you try to reboot tor on your mynode and also force quit and reopen FullyNoded (forces tor to reboot on FN) and see if it resolves your issue?

aetikon commented 2 years ago

We rebooted MyNode and force closed FN on the iPhone, but that didn't resolve the problem. MyNode is fully updated (bitcoin core 22.0.0 and LND 0.13.3)

We also configured the iPhone to connect to MyNode's VPN and attempted to connect FN to LND directly using REST + Local IP, but the connection failed with "The operation couldn't be completed. Invalid argument."

Fonta1n3 commented 2 years ago

We rebooted MyNode and force closed FN on the iPhone, but that didn't resolve the problem. MyNode is fully updated (bitcoin core 22.0.0 and LND 0.13.3)

We also configured the iPhone to connect to MyNode's VPN and attempted to connect FN to LND directly using REST + Local IP, but the connection failed with "The operation couldn't be completed. Invalid argument."

https://github.com/Fonta1n3/FullyNoded/commit/a87d5e74dc2a3c0feed475b1999cb20ea98d1ddc should fix this, if you can build from source and test that'd be great. Tor only for now, local connection coming soon.

Fonta1n3 commented 2 years ago

For the record I can't test as I do not have a mynode, for my Nodl the old code and this code works fine.

Fonta1n3 commented 2 years ago

v0.2.28 should also work with localhost for LND and c-lightning 👍🏻

aetikon commented 2 years ago

That's great news. Thank you for the quick turnaround on this. I could build this but I also don't have access to the mynode in question.