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.

Sign up for GitHub

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

Jump to bottom

P2P: add wrapper/abstraction for known peers #2188

Open

CHr15F0x opened this issue Aug 26, 2024 · 0 comments

Milestone

P2P peer management

Member

CHr15F0x commented Aug 26, 2024 •

edited

Loading

Instead of referring directly to libp2p::kad for known peers consider creating a wrapper that would hold the list of known peers.
The "list" would:

include "in-config" predefined peers (which ofc should also be in the local DHT)
be in sync with any kad updates (in particular new peers)

Pros:

syncing would proceed immediately provided that at least one peer was predefined via config,
internal p2p client logic wouldn't have to periodically poll for new/random peers.

Also see: #2187

CHr15F0x mentioned this issue

P2P: update bootstrap logic, remove explicit triggering #2189

Closed

CHr15F0x added this to the P2P peer management milestone

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