Dmitry Markin
Dmitry Markin
> Ended it up going in a slightly different direction on this PR and decided that the right fix here is to add a signed creation_time to `SignedAuthorityRecord` This seems...
@alexggh I see @lexnv has already addressed 1) in https://github.com/paritytech/litep2p/pull/76. As for 2), it should be not that difficult to implement `put_record_to` as we need to only contact one specific...
It doesn't seem that this is happening due to 1. or 2. Validation slots are always deallocated once validations complete here: https://github.com/paritytech/polkadot-sdk/blob/21b32849db4ac5f284d7ca2f93927b0d35bb783a/substrate/client/network/sync/src/block_announce_validator.rs#L297 and per-peer slot deallocation is tested here: https://github.com/paritytech/polkadot-sdk/blob/21b32849db4ac5f284d7ca2f93927b0d35bb783a/substrate/client/network/sync/src/block_announce_validator.rs#L359-L383...
It looks like the issue might be caused by major syncing peers announcing imported blocks. By default, we always announce imported blocks: https://github.com/paritytech/polkadot-sdk/blob/8ce16ee6ff190912995dc6c8da85b741eaada395/substrate/client/service/src/lib.rs#L184-L186 `announce_imported_blocks`: https://github.com/paritytech/polkadot-sdk/blob/8ce16ee6ff190912995dc6c8da85b741eaada395/substrate/client/cli/src/config.rs#L431-L436 There is no "fuse" in...
https://github.com/paritytech/polkadot-sdk/pull/2132 helped with polling block announce validations during versi scaling, but if the issue described in https://github.com/paritytech/polkadot-sdk/issues/2152 continues, we'll need to look into it again.
> We should probably stop logging this, as this could be literally someone spamming the node. Addressed in https://github.com/paritytech/polkadot-sdk/pull/4480.
This is blocked by breaking changes in multiaddr, multihash, libp2p-identity and should be addressed for all these libraries together.
And likely later we will be forced to fork `multiaddr` and make it not depend on `libp2p`.
`PeerIdMismatch(PeerId, PeerId)` this one is used by https://github.com/dmitry-markin/p2p-probe to detect the correct `PeerId`.
Currently ignored clippy lints for context: https://github.com/paritytech/litep2p/blob/5c37774d0b377430bad408a6f7ed5e9f1ff53c7f/src/lib.rs#L21-L29