Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

misc(p2p/peerTracker): peerTracker improvement #160

Open
vgonkivs opened this issue Feb 26, 2024 · 2 comments
Open

misc(p2p/peerTracker): peerTracker improvement #160

vgonkivs opened this issue Feb 26, 2024 · 2 comments

Comments

@vgonkivs
Copy link
Member

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

  • fix(p2p/session): return peer to the queue in case of ErrNotFound #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
Copy link
Member

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
Copy link
Member Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants