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.

Sign up for GitHub

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

Jump to bottom

External gateway detection seems off #109

Closed
whyrusleeping opened this issue Sep 13, 2016 · 3 comments
Closed

External gateway detection seems off #109

whyrusleeping opened this issue Sep 13, 2016 · 3 comments
Labels
need/community-input Needs input from the wider community research
Milestone
Improve Connectivity

Comments

@whyrusleeping
Copy link
Contributor

Several users have reported (and I have noticed occasionally) that the addresses we advertise for other peers to connect to us on arent quite right. For example, peers might see an address with our correct external address, but without the right external port. We should investigate and test this.
@whyrusleeping whyrusleeping added this to the Improve NAT Traversal milestone Sep 13, 2016
@whyrusleeping whyrusleeping added the status/deferred Conscious decision to pause or backlog label Sep 14, 2016
@jbenet
Copy link
Contributor

jbenet commented Sep 15, 2016

NAT mapping may have changed in the time.

@bigs bigs added research need/community-input Needs input from the wider community labels Sep 11, 2018
@bigs
Copy link
Contributor

bigs commented Sep 11, 2018

reopen if we want to re-approach this

@bigs bigs closed this as completed Sep 11, 2018
@ghost ghost removed the status/deferred Conscious decision to pause or backlog label Sep 11, 2018
@Stebalien
Copy link
Member

Replaced by #383 with more details.

marten-seemann added a commit that referenced this issue Dec 20, 2021
@marten-seemann
Merge pull request #109 from libp2p/clean-shutdown
8cc74d3 
remove context from constructor, implement a proper Close method
marten-seemann added a commit that referenced this issue Apr 22, 2022
@marten-seemann
Merge pull request #109 from libp2p/rcmgr
2d29093 
use the resource manager
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
need/community-input Needs input from the wider community research
Projects
None yet
Milestone
Improve Connectivity
Development

No branches or pull requests
5 participants
@jbenet @bigs @Stebalien @daviddias @whyrusleeping