Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to connect to LND using IP address #17

Open
yzernik opened this issue Jul 8, 2022 · 6 comments
Open

Unable to connect to LND using IP address #17

yzernik opened this issue Jul 8, 2022 · 6 comments

Comments

@yzernik
Copy link

yzernik commented Jul 8, 2022

I get an error when I try to connect to my local LND using http://127.0.0.1:10002:

ConnectError { internal: Connect { address: \\\"http://127.0.0.1:10002\\\", error: tonic::transport::Error(Transport, hyper::Error(Connect, InvalidDNSNameError)) }

But it works when I use http://localhost:10002.

@yzernik
Copy link
Author

yzernik commented Jul 8, 2022

This appears to be an issue in the rustls library:
rustls/rustls#184

@yzernik
Copy link
Author

yzernik commented Jul 13, 2022

I created a fork of this repo using openssl instead of rustls as the TLS library: https://github.com/yzernik/tonic_openssl_lnd

My fork allows me to connect to an LND instance even when the host is an IP address.

I published it as a crate if anyone else wants to use it: https://crates.io/crates/tonic_openssl_lnd

@grunch
Copy link
Contributor

grunch commented Aug 4, 2022

Hi, why don't you make PR instead? I think it's better for all the tonic_lnd users

@Kixunil
Copy link
Owner

Kixunil commented Aug 4, 2022

This actually looks like missing s, should've been https://, not http.

And sorry for replying this late, I saw the message but couldn't reply right way and it fell out of my notifications. :(

@grunch
Copy link
Contributor

grunch commented Aug 4, 2022

Oh he did the PR, let's review it I need this to be merged 🥺

@Kixunil
Copy link
Owner

Kixunil commented Aug 5, 2022

It's quite large, I will need more time to review. :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants