Skip to content
This repository has been archived by the owner on Mar 9, 2022. It is now read-only.

libp2p v2 relay functionality is on by default and configurable #147

Closed
BigLep opened this issue Oct 5, 2021 · 3 comments · Fixed by #146
Closed

libp2p v2 relay functionality is on by default and configurable #147

BigLep opened this issue Oct 5, 2021 · 3 comments · Fixed by #146
Assignees

Comments

@BigLep
Copy link

BigLep commented Oct 5, 2021

Done criteria: go-libp2p's v2 relay functionality is on by default but configurable. Accompanying docs are present.

Why important: Project Flare will be most successful if we can have sufficient relays present on the network so that there is a large pool of nodes to perform relaying.

Customer:

Notes: relaying nodes won't be advertised to the DHT. That is a future effort.

@aschmahmann
Copy link
Contributor

aschmahmann commented Oct 5, 2021

This seems 👍. The one exception I'd add here is that relaying should not be on for nodes that themselves are NAT'd/require a relay. I suspect go-libp2p will take care of this for us, but in case it doesn't it's something we'll have to consider when dealing with the configurability and what the default mode will be. This seems similar in spirit to the DHT's auto mode.

IIUC the relaying we're talking about here is configurable v2 relaying where the amount of bandwidth donated by peers is fairly minimal. Out of curiosity how difficult is it to setup a few default relays to test the protocol in production first before widely deploying it? If each peer use of the relay consumes minimal bandwidth then using a few centralized relays for the network to start might not be such a big deal. However, I might be drastically underestimating the scales here.

@BigLep BigLep changed the title libp2p relay functionality is on by default and configurable libp2p v2 relay functionality is on by default and configurable Oct 6, 2021
@BigLep BigLep transferred this issue from ipfs/kubo Oct 19, 2021
@welcome
Copy link

welcome bot commented Oct 19, 2021

Thank you for submitting your first issue to this repository! A maintainer will be here shortly to triage and review.
In the meantime, please double-check that you have provided all the necessary information to make this process easy! Any information that can help save additional round trips is useful! We currently aim to give initial feedback within two business days. If this does not happen, feel free to leave a comment.
Please keep an eye on how this issue will be labeled, as labels give an overview of priorities, assignments and additional actions requested by the maintainers:

  • "Priority" labels will show how urgent this is for the team.
  • "Status" labels will show if this is ready to be worked on, blocked, or in progress.
  • "Need" labels will indicate if additional input or analysis is required.

Finally, remember to use https://discuss.ipfs.io if you just need general support.

@BigLep BigLep linked a pull request Oct 19, 2021 that will close this issue
@BigLep BigLep moved this from In Progress to In Review in Go IPFS Roadmap Oct 19, 2021
@vyzo
Copy link
Contributor

vyzo commented Oct 19, 2021 via email

Go IPFS Roadmap automation moved this from In Review to Done Nov 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants