paritytech / polkadot-sdk

The Parity Polkadot Blockchain SDK
https://polkadot.com/
1.92k stars 707 forks source link

network: Authoriy discovery records published without IP addresses #6518

Open lexnv opened 1 week ago

lexnv commented 1 week ago

We have an unusual number of warnings from litep2p running 0.6.2 in kusama as validators.

The warnings are hamless, however they expose an underlying issue wrt publishing authority discovery records in DHT:

WARN tokio-runtime-worker sub-libp2p: couldn't add known address (/p2p/12D3KooWFie68KdEbjkqtHgzkWA4MfbnUYipqBQHXtheydcuPKG7) for PeerId("12D3KooWFie68KdEbjkqtHgzkWA4MfbnUYipqBQHXtheydcuPKG7"), unsupported transport

Using subp2p-explorer tool to discover authorities, some authorities are publishing empty addresses of the form: /p2p/[peer ID]:

authority="CzHcp6nEsT4NJuAbf4yCS2J8KKYpvuAe63bkWrPWhjaaT6w" 

peer_id=PeerId("12D3KooWFegWJubWCeJCbWpDEAwukQmXKaWTyRgRRZ8NNdzmuxSr")

version="Parity Polkadot/v1.16.1-835e0767fe8 (staked-eu-north-1-472-a)"

addresses={.., /p2p/12D3KooWFegWJubWCeJCbWpDEAwukQmXKaWTyRgRRZ8NNdzmuxSr, ..}

The previous version corresponds to `v1.16.1-rc2.

cc @paritytech/networking