[go: up one dir, main page]

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

docs: add note about webtransport only supporting dial #2411

Merged
merged 3 commits into from
Feb 19, 2024

Conversation

Download
Copy link
Contributor
@Download Download commented Feb 12, 2024

Title

docs: add note about webtransport only supporting dial

Description

This adds a note at the top of the WebTransport README.md to remind people of the fact that currently, the WebTransport implementation in js-libp2p only supports dialing out to other nodes and does not actually support listening for incoming dials because js-libp2p depends on Node JS and QUIC support has not landed in Node JS yet.

Related: #2371

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if necessary (this includes comments as well)

This adds a note at the top of the WebTransport `README.md` to remind people of the fact that currently, the WebTransport implementation in js-libp2p only supports dialing out to other nodes and does not actually support listening for incoming dials because js-libp2p depends on Node JS and QUIC support has not landed in Node JS yet.

Related: libp2p#2371
@Download Download requested a review from a team as a code owner February 12, 2024 10:47
@Download
Copy link
Contributor Author

@achingbrain Re-created this pr according to your instructions.

@achingbrain
Copy link
Member

Thanks!

@achingbrain achingbrain merged commit 8072a2e into libp2p:main Feb 19, 2024
3 checks passed
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.

2 participants