feat: improve relay announce and bootstrap handling#343
Closed
NiKrause wants to merge 1 commit into
Closed
Conversation
8d915b0 to
27d5549
Compare
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
This PR improves relay address publication and browser bootstrap handling:
LIBP2P_ANNOUNCE_ADDRSgo-peerjs-peerprefer explicitly configured browser-dialable relay bootstrap addresses when presentWhy
Some deployments expose externally reachable relay addresses that should be announced explicitly rather than inferred locally.
In practice, a relay may have multiple useful public transport paths, but browser clients still need a clear and reliable bootstrap address set. This PR improves how the UC relay publishes those addresses and how
js-peerconsumes browser-dialable bootstrap addresses.This also helps deployment environments such as Aleph Cloud, where externally reachable addresses are assigned outside the guest and need to be injected into relay announcements, but the main goal of this PR is runtime/bootstrap correctness rather than deployment workflow changes.
The dependency refresh is included because the newer
go-libp2p/p2p-forgestack is required by the current AutoTLS integration and authentication flow.Scope
This PR intentionally stays focused on runtime/connectivity behavior:
go-peer/main.gogo-peer/go.modgo-peer/go.sumjs-peer/src/lib/libp2p.tsjs-peer/.env.exampleIt does not include the Aleph workflow stack, deploy actions, or fork-specific operational wiring.
Validation
go test ./...ingo-peer