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

Peer discovery & management #52

Closed
hopeyen opened this issue Mar 27, 2023 · 2 comments · Fixed by #85
Closed

Peer discovery & management #52

hopeyen opened this issue Mar 27, 2023 · 2 comments · Fixed by #85
Assignees
Labels
enhancement New feature or request

Comments

@hopeyen
Copy link

hopeyen commented Mar 27, 2023

At the moment, enabling DiscV5 mechanism requires multiaddresses upon node handle initialization, while DNS discovery takes enrtree url and resolves to multiaddresses after node initialization.

It would be nice if the two mechanisms are consistent, potentially

  1. DiscV5 can be initialized with a ENRtree url instead of a set of multiaddresses
  2. DiscV5 can be enabled after the initialization of the node
  3. Or if an ENRtree url can be optionally supplied with a toggle for discovery mechanisms, between static list, DNS discovery, DiscV5, ...

A potential flow is

  1. Initialize waku node, use DNS url to get bootstrap nodes
  2. Connect to bootstrap nodes either by DNS discovery or enable DiscV5
  3. Remember "good" peers to use in DiscV5 when node gets restarted
@harsh-98
Copy link

harsh-98 commented Nov 13, 2023

Weekly Update

  • achieved: Worked mainly on rust-waku-bindings issues for go-waku.
  • next: Handle epic 2.2

@richard-ramos
Copy link
Member

Weekly Update

  • achieved: Added dns discovery parameters to the node config

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants