-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
autorelay #454
Merged
Merged
autorelay #454
Changes from all commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
4ea04c2
identify: implement identify push protocol
vyzo a89e74a
basic host: export AddrsFactory, provide method to push identify
vyzo a585891
autorelay hosts
vyzo 9795a01
configurable boot delay for autorelay
vyzo 5c623f2
name the autorelay logger as such
vyzo d24fe6a
update libp2p.New constructor to construct relayed/routed hosts
vyzo 6bdfcac
use AllAddrs as the address factory in autonat
vyzo 0ff7393
fix announced relay address
vyzo 477c19a
reduce boot delay to 60s
vyzo 7fabe39
parallel identify push
vyzo c6d1eeb
autorelay test
vyzo f6c38c9
filter unspecific relay address
vyzo 8d108dd
import go-libp2p-discovery and go-libp2p-autonat
vyzo 8faf540
fix typo
vyzo b62c83b
better limit adjustment for relay discovery
vyzo ee69383
extend autorelay test to verify connectivity
vyzo dcb8fd2
fix inverted address selection logic in test
vyzo 89aef89
don't adveretise unspecific relay addrs in RelayHost
vyzo a7e1bf0
call the routing interface BasicRouting, alias to top level type
vyzo 2993fd9
add autorelay documentation
vyzo cc2dd22
make randezvous key a constant
vyzo 67aba4d
move relay selection strategy out of line
vyzo 4f90393
add a comment
vyzo 84400e2
fix typo
vyzo e96605d
use /p2p multiaddr
vyzo 4be7ada
tag relay connections
vyzo fdfa224
document doUpdateAddrs
vyzo 292b8a1
update gx deps
vyzo c2d846a
fix go vet issue
vyzo 7b324b1
remove BasicRouting interface; use PeerRouting and upcast for discovery
vyzo 0dfca3b
increase test AutoNATIdentifyDelay to 100ms
vyzo 69144bd
extend autorelay test to verify pushing of relay addrs
vyzo 5d8988f
add comment about the unstable nature of BasicHost.PushIdentify
vyzo 5a1c09c
use advertised addrs for autonat dial back, not all addrs
vyzo 4cb5d00
gx update
vyzo a309f09
Add note to relay docs about internal interface instability
vyzo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add a comment saying that this is an unstable interface. Ideally, we'd have an address manager, a protocol muxer, etc. that all emit events. Then, the identify service would subscribe to these events to determine when and if to push information to peers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, will add a comment.