Skip to content

Swarm + Handshake Listen Addresses #271

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

Merged
merged 4 commits into from
Nov 5, 2014
Merged

Swarm + Handshake Listen Addresses #271

merged 4 commits into from
Nov 5, 2014

Conversation

jbenet
Copy link
Member

@jbenet jbenet commented Nov 5, 2014

This sets up the handshake to update the local peer
with remote observed addresses. this way we can continue
to advertise to others candidate addresses observed by
others.

(this won't be a good way to go in the long run
because anyone can lie. we'll have to verify those
reported addresses are good candidates, etc.)

The check needed knowledge of the _listen_ addresses,
not just the interface addresses. Also, the handshake now
sends out all the addresses we accumulate about ourselves.
(this may be bad in the long run, but useful now to test)
@jbenet jbenet added the status/in-progress In progress label Nov 5, 2014
@jbenet
Copy link
Member Author

jbenet commented Nov 5, 2014

This will help with nat traversal things

jbenet added a commit that referenced this pull request Nov 5, 2014
Swarm + Handshake Listen Addresses
@jbenet jbenet merged commit 7cfe2f1 into master Nov 5, 2014
@jbenet jbenet deleted the listen-addrs branch November 5, 2014 17:25
@jbenet jbenet removed the status/in-progress In progress label Nov 5, 2014
@whyrusleeping
Copy link
Member

Maybe we should attempt a connection to the address given and check that we are connecting to ourselves?

@Stebalien Stebalien mentioned this pull request May 26, 2020
77 tasks
@ajnavarro ajnavarro mentioned this pull request Aug 24, 2022
72 tasks
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