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

Missing addr flag for announcing core-lightning nodes externally #608

Merged
merged 1 commit into from
Oct 13, 2022

Conversation

niteshbalusu11
Copy link
Contributor

Description

The missing --addr=0.0.0.0:9735 causes failure in connecting to core-lightning nodes externally. Polar in the connect tab shows P2P External for core-lightning nodes but it does not work because it's not being announced.

Steps to Test

  1. Try connecting to the p2p external port using telnet, for example telnet localhost 9836 without the announcing, connection succeeds but terminates instantly.
  2. Announce the port, restart the node and try connecting again. Connection persists.

You can also try testing using something like this library https://github.com/jb55/lnsocket/tree/master/examples and commando runes.

The missing `--addr=0.0.0.0:9735` causes failure in connecting to core-lightning nodes externally. Polar in the connect tab shows P2P External for core-lightning nodes but it does not work because it's not being announced.
@niteshbalusu11 niteshbalusu11 changed the title Missing address flag for announcing externally Missing addr flag for announcing code-lightning nodes externally Oct 12, 2022
@niteshbalusu11 niteshbalusu11 changed the title Missing addr flag for announcing code-lightning nodes externally Missing addr flag for announcing core-lightning nodes externally Oct 12, 2022
Copy link
Owner

@jamaljsr jamaljsr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK LGTM 👍

Thanks for the fix

@jamaljsr jamaljsr merged commit b011dfc into jamaljsr:master Oct 13, 2022
@niteshbalusu11 niteshbalusu11 deleted the patch-1 branch October 13, 2022 12:26
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 this pull request may close these issues.

2 participants