Skip to content

[stable2506] Backport #9281#9329

Merged
EgorPopelyaev merged 3 commits intostable2506from
backport-9281-to-stable2506
Jul 31, 2025
Merged

[stable2506] Backport #9281#9329
EgorPopelyaev merged 3 commits intostable2506from
backport-9281-to-stable2506

Conversation

@paritytech-release-backport-bot
Copy link
Copy Markdown

Backport #9281 into stable2506 from lexnv.

See the documentation on how to use this bot.

…ternal (#9281)

This PR ensures that external addresses discovered by the identify
protocol are not propagated to the litep2p backend if they are not
global. This leads to a healthier DHT over time, since nodes will not
advertise loopback / non-global addresses.

We have seen various cases were loopback addresses were reported as
external:

```
2025-07-16 16:18:39.765 TRACE tokio-runtime-worker sub-libp2p::discovery: verify new external address: /ip4/127.0.0.1/tcp/30310/p2p/12D3KooWNw19ScMjzNGLnYYLQxWcM9EK9VYPbCq241araUGgbdLM

2025-07-16 16:18:39.765  INFO tokio-runtime-worker sub-libp2p: 🔍 Discovered new external address for our node: /ip4/127.0.0.1/tcp/30310/p2p/12D3KooWNw19ScMjzNGLnYYLQxWcM9EK9VYPbCq241araUGgbdLM
```

This PR takes into account the network config for
`allow_non_global_addresses`.

Closes: #9261

cc @paritytech/networking

---------

Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 7a60803)
@github-actions
Copy link
Copy Markdown
Contributor

This pull request is amending an existing release. Please proceed with extreme caution,
as to not impact downstream teams that rely on the stability of it. Some things to consider:

  • Backports are only for 'patch' or 'minor' changes. No 'major' or other breaking change.
  • Should be a legit fix for some bug, not adding tons of new features.
  • Must either be already audited or not need an audit.
Emergency Bypass

If you really need to bypass this check: add validate: false to each crate
in the Prdoc where a breaking change is introduced. This will release a new major
version of that crate and all its reverse dependencies and basically break the release.

@EgorPopelyaev EgorPopelyaev enabled auto-merge (squash) July 31, 2025 13:58
@EgorPopelyaev EgorPopelyaev merged commit b2950c2 into stable2506 Jul 31, 2025
253 of 258 checks passed
@EgorPopelyaev EgorPopelyaev deleted the backport-9281-to-stable2506 branch July 31, 2025 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A3-backport Pull request is already reviewed well in another branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants