misc(p2p/peerTracker): peerTracker improvement

celestiaorg / go-header

Go library with all the services needed to request, sync and store blockchain headers.

Apache License 2.0

19 stars 16 forks source link

Open vgonkivs opened 8 months ago

vgonkivs commented 8 months ago

Last week we discussed with @Wondertan ways to improve the peer tracker:

https://github.com/celestiaorg/go-header/pull/159 introduces adding peers who are currently syncing back to the queue to request them one more time. We should add a backoff for these peers;
move scoring to the Connection Manager under a specific tag;
remove limiting for 100 tracked peers in the peer tracker;
check peer support for the protocol. We should migrate from Connect event to Identify.

Wondertan commented 8 months ago

To finish off this, we would need to add test that ensures we can get valid headers from a single honest peers, when other hundred is malicious.

vgonkivs commented 8 months ago

Agree. This test case will be added in scope of the pr