Skip to content

refactor(libp2p): rename underlays to overlayToPeerID in peer registry - #5337

Merged
gacevicljubisa merged 1 commit into
masterfrom
refactor/peer-registry-underlays-to-peer-ids
Jan 29, 2026
Merged

refactor(libp2p): rename underlays to overlayToPeerID in peer registry#5337
gacevicljubisa merged 1 commit into
masterfrom
refactor/peer-registry-underlays-to-peer-ids

Conversation

@gacevicljubisa

@gacevicljubisa gacevicljubisa commented Jan 29, 2026

Copy link
Copy Markdown
Member

Checklist

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

The field 'underlays' was misleadingly named as it:

  • Is indexed by overlay addresses, not underlay addresses
  • Stores libp2p peer IDs, not underlay network addresses
  • Used plural form suggesting multiple values per key

Renamed to 'overlayToPeerID' to accurately reflect that it maps
overlay addresses (string) to their corresponding libp2p peer IDs.

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

@gacevicljubisa gacevicljubisa changed the title refactor(libp2p): rename underlays to peerIDsByOverlay in peer registry refactor(libp2p): rename underlays to overlayToPeerID in peer registry Jan 29, 2026
@gacevicljubisa
gacevicljubisa merged commit 6ddf9b4 into master Jan 29, 2026
19 checks passed
@gacevicljubisa
gacevicljubisa deleted the refactor/peer-registry-underlays-to-peer-ids branch January 29, 2026 12:07
v1rtl added a commit to v1rtl/bee that referenced this pull request Jun 2, 2026
Relax the non-empty underlay validation in bzz.ParseAddress/MarshalJSON so
browser/wasm and strict-NAT peers that cannot be dialed back can still
handshake and use protocols over existing connections. Guard the Connect
addressbook.Put so such peers are not stored for reconnection (v2.8 already
guards handleIncoming and the reacher). Port of pr-5326; peer registry
rename already landed upstream via ethersphere#5337.

Note: this is a deliberate protocol relaxation vs upstream v2.8.
v1rtl added a commit to v1rtl/bee that referenced this pull request Jun 2, 2026
Relax the non-empty underlay validation in bzz.ParseAddress/MarshalJSON so
browser/wasm and strict-NAT peers that cannot be dialed back can still
handshake and use protocols over existing connections. Guard the Connect
addressbook.Put so such peers are not stored for reconnection (v2.8 already
guards handleIncoming and the reacher). Port of pr-5326; peer registry
rename already landed upstream via ethersphere#5337.

Note: this is a deliberate protocol relaxation vs upstream v2.8.
v1rtl added a commit to v1rtl/bee that referenced this pull request Aug 10, 2026
Relax the non-empty underlay validation in bzz.ParseAddress/MarshalJSON so
browser/wasm and strict-NAT peers that cannot be dialed back can still
handshake and use protocols over existing connections. Guard the Connect
addressbook.Put so such peers are not stored for reconnection (v2.8 already
guards handleIncoming and the reacher). Port of pr-5326; peer registry
rename already landed upstream via ethersphere#5337.

Note: this is a deliberate protocol relaxation vs upstream v2.8.
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.

3 participants