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

feat: add native Webtransport transport #4348

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mxinden
Copy link
Member

@mxinden mxinden commented Aug 19, 2023

Description

Work in progress

Here to prevent two implementations to emerge in parallel.

Notes & open questions

Fixes #2993.

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • A changelog entry has been made in the appropriate crates

@oblique
Copy link
Contributor

oblique commented Oct 14, 2023

I'm wondering. With the current libp2p API, will it be possible to have Quic and WebTransport listening on the same port?

@thomaseizinger
Copy link
Contributor

I'm wondering. With the current libp2p API, will it be possible to have Quic and WebTransport listening on the same port?

I don't think we will able to do that straight away. We need to share a UDP socket between the two for that. I've been thinking about how to offer this and it might be possible if we allow to pass some (shared) abstraction over a socket into each transport.

@mxinden
Copy link
Member Author

mxinden commented Oct 16, 2023

I'm wondering. With the current libp2p API, will it be possible to have Quic and WebTransport listening on the same port?

@oblique in case it is of some interest, here is the corresponding work on the go-libp2p side libp2p/go-libp2p#1759.

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

Successfully merging this pull request may close these issues.

transport/webtransport: Add WebTransport protocol
3 participants