Don't require TLS connection when connecting to localhost or onion addresses

[Kixunil]

When an address matching regex ^(localhost|127.0.0.1|.*\.onion)(:[1-9][0-9]*)?$ is entered, show a checkbox "Disable TLS", as the encryption might not be needed.

This is useful for example when one sets up an SSH tunnel using ConnectBot.

[tomichec]

the required change can be done here: https://github.com/ACINQ/phoenix/blob/3ed8cae1f32d89e44a00236792e3e92a311c7b9d/app/src/main/java/fr/acinq/phoenix/lnurl/LNUrl.kt#L67

[Kixunil]

That looks like LNURL, I was talking about connecting to a custom node.

[dpad85]

Hi, from v1.3.2 you can disable/enable TLS when connecting to an onion electrum server (it's disabled by default) so this issue is half fixed. What needs to be done is to add the same behaviour for localhost|127.0.0.1 addresses.

[tomichec]

It would also be useful to have this behavior for lnurl.

[slush0]

Please allow connecting to non-ssl electrum nodes. There are setups when it is completely unnecessary or even impossible to fulfill this requirement. I'm using Electrum server in private network behind VPN and it is very hard to generate ssl cert for publicly unreachable machines.

[Kixunil]

It would also be useful to have this behavior for lnurl.

For onion only, I don't believe LNURL is useful for localhost queries.

@slush0 good point about VPN, that'd be hard to check correctly as I suggested. Maybe allow it with big fat warning?

[slush0]

Any progress on this? Custom servers to require SSL is really limiting. Please allow to skip this requirement.