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

Investigate external/internal ip for network discovery. #82

Closed
dj8yfo opened this issue Apr 28, 2017 · 3 comments
Closed

Investigate external/internal ip for network discovery. #82

dj8yfo opened this issue Apr 28, 2017 · 3 comments
Labels

Comments

@dj8yfo
Copy link
Contributor

dj8yfo commented Apr 28, 2017

The fact, that node is sending initially statically defined addr of itself in Connect message may be problematic for deployment of nodes across different networks/organisations.
In general case, a node cannot know its own ip, as seen by another peer, without using external services.
Moreover, a node's ip may vary from different peers' perspective.
#16
https://*********/projects/22/tasks/1307

@defuz defuz added the icebox label May 1, 2017
@vldm
Copy link
Contributor

vldm commented Sep 6, 2017

For now #214, we have different external and listener ip in config, this should partially helps to deploy node in WAN.

For now, we should split this task into two sub cases:

  • Node can work with more than one interface and ip respectively.
  • There should be some mechanism to resolve external ip

Additionally there some problems revealed during discussion:

  1. Sometimes auditor node could be outside internal subnet, this allows validator's peer-exchange to reveal auditor ip to other validators. May be auditors should be outside peer-exchange mechanism.
  2. With more than one external ip, there should be feedback from outgoing connections, to check if we connect to valid validator.

@alekseysidorov
Copy link
Contributor

As workaround we can define external addresses transformation table with the such format:

[peer_external_adresses]
"peer_pubkey" = "ip_addr_in_our_subnet"

stanislav-tkach added a commit to stanislav-tkach/exonum that referenced this issue Feb 10, 2018
@stanislav-tkach
Copy link
Contributor

Closed by #406.

stanislav-tkach pushed a commit to stanislav-tkach/exonum that referenced this issue Apr 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

5 participants