Skip to content

[unstable2507] Backport #9281#9330

Merged
EgorPopelyaev merged 3 commits intounstable2507from
backport-9281-to-unstable2507
Jul 30, 2025
Merged

[unstable2507] Backport #9281#9330
EgorPopelyaev merged 3 commits intounstable2507from
backport-9281-to-unstable2507

Conversation

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

Backport #9281 into unstable2507 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)
@EgorPopelyaev EgorPopelyaev enabled auto-merge (squash) July 30, 2025 12:54
@paritytech-workflow-stopper
Copy link
Copy Markdown

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/16623099686
Failed job name: run-frame-omni-bencher

@EgorPopelyaev EgorPopelyaev merged commit 67d894d into unstable2507 Jul 30, 2025
240 of 241 checks passed
@EgorPopelyaev EgorPopelyaev deleted the backport-9281-to-unstable2507 branch July 30, 2025 15:24
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