P2P: checkpoint sync fails on a malicious peer

eqlabs / pathfinder

A Starknet full node written in Rust

https://eqlabs.github.io/pathfinder/

Other

613 stars 222 forks source link

Open CHr15F0x opened 3 weeks ago

CHr15F0x commented 3 weeks ago

Right now checkpoint sync does not retry if it encounters an error, neither does it punish the peer that produces data that failed verification.

Checkpoint sync should:

punish the offending peer, just similarly to what the p2p client does,
retry upon reaching an error related to malicious peer/data, the p2p client will then reach out to the current set of peers minus the last offending peer.

CHr15F0x commented 2 weeks ago

Removing the bug label because it's a missing feature not a real bug.