lightning-browser-extension icon indicating copy to clipboard operation
lightning-browser-extension copied to clipboard

Add support for lndconnect URLs

Open bumi opened this issue 4 years ago • 3 comments

lnd connect describes a URL that contains all need information to connect to an lnd node.

it uses the lndconnect: scheme. We should add support for those links and automatically prefill the connector setting to make it easy for users to add an LND connection.

https://github.com/LN-Zap/lndconnect/blob/master/lnd_connect_uri.md

bumi avatar Oct 06 '21 16:10 bumi

Wouldn't mind doing some pair programming on this if anyone is interested.

secondl1ght avatar Nov 02 '21 03:11 secondl1ght

I basically got this working, but the problem is that most LND nodes (like Umbrel) use TLS with a self-signed certificate. There is no way in the fetch API to ignore the certificate errors from this AFAIK. I think it would ork over Tor. I'll open a draft PR soon, maybe someone has an idea.

AaronDewes avatar Feb 13 '22 09:02 AaronDewes

ah true. Sadly we have to rely on the browser for the certificates. This means similar to Tor requests can be made through the native companion app, OR: an exception has to be added or the certificate has to be added to the browser. Mostly it should be actually enough to open the URL once manually and add an exception.

for loading lndconnect strings we should probably ignore the certificate for now.

bumi avatar Feb 14 '22 11:02 bumi

I believe this is implemented already, so closing

stackingsaunter avatar Mar 16 '23 13:03 stackingsaunter