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

ROADMAP.md: Mark hole punching on TCP and QUIC as done #415

Merged
merged 2 commits into from
May 24, 2022

Conversation

mxinden
Copy link
Member

@mxinden mxinden commented May 20, 2022

  • Introduce "Done" section.
  • Reduce scope of "Unprecedented global connectivity" to hole punching on QUIC
    and TCP.
  • Move item down to "Done" section.

Browser connectivity is tracked via #407 and #414. I will add an additional pull request to track stream migration. Testing will be tracked via "Automated compatibility testing" section.

As far as I remember, this is what we agreed on in Paris. Please correct me in case I am wrong.

- Introduce "Done" section.
- Reduce scope of "Unprecedented global connectivity" to hole punching on QUIC
  and TCP.
- Move item down to "Done" section.
@mxinden mxinden merged commit 85fa69a into libp2p:master May 24, 2022
@UnkwUsr
Copy link

UnkwUsr commented Apr 24, 2023

@mxinden
this commit breaks link #unprecedented-global-connectivity in the same file: (lines pointed)

specs/ROADMAP.md

Lines 549 to 551 in f8f32f7

- **Connectivity-related problems:** see [Unprecedented global
connectivity](#unprecedented-global-connectivity) and
[Improve NAT traversal](#improve-nat-traversal).

P.S. next link there (#improve-nat-traversal) also broken, but seems it was this from birth (99a4319)

@mxinden
Copy link
Member Author

mxinden commented Apr 26, 2023

@UnkwUsr thanks for reporting. Mind submitting a pull request?

@UnkwUsr
Copy link

UnkwUsr commented Apr 26, 2023

@mxinden no, I'm not very confident with ipfs specs, so not sure how it should looks, hence I sent this comment to point out just that something broken. Feel free to commit/submit pull request :). Thanks

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

Successfully merging this pull request may close these issues.

3 participants