Ride-The-Lightning / RTL

Ride The Lightning - A full function web browser app for LND, C-Lightning and Eclair
MIT License
749 stars 160 forks source link

Loop-in HTLC window needs to specify that it is local LND wallet only. #363

Open beeforbacon opened 4 years ago

beeforbacon commented 4 years ago

In RTL loop-in section at the end where it exposes the HTLC address should specify that this address is for local LND loop-in transactions only. I made the mistake of sending external funds to the address thinking I could fund externally. As far as I can tell, there is no setting within RTL (although there is in the lncli) to specify external funding. Until external funding is enabled, please add a warning not to send from external wallet to the address. I am sure more may make the mistake.

saubyk commented 4 years ago

Hi @beeforbacon looking at this issue now. Are you sure, it's for 'Loop In'? AFAIK, Loop In does not display any address in RTL.

However Loop Out does display an HTLC address at the end. Which is definitely not meant to be used to send any kind of funds to. It's returned by the Loop Out API. We can hide this address, to ensure that it does not causes any confusion.

ShahanaFarooqui commented 4 years ago

@saubyk I need more clarification/details for any further action.