-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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(unstable): WebTransport #27431
base: main
Are you sure you want to change the base?
feat(unstable): WebTransport #27431
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will take a deeper look after holiday break.
Can you please open a PR to https://github.com/mdn/browser-compat-data that adds proper information about Deno having this API supported?
pub(crate) mod webtransport { | ||
// MIT License | ||
// | ||
// Copyright (c) 2023 Luke Curley | ||
// | ||
// Permission is hereby granted, free of charge, to any person obtaining a copy | ||
// of this software and associated documentation files (the "Software"), to deal | ||
// in the Software without restriction, including without limitation the rights |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason to vendor this code? If so can you please add a link here to the original code?
Initial implementation of WebTransport client and server!
This is very unstable because the interface should eventually shift to use hyper (h3 is on the 2025 roadmap) instead of manually messing with the the protocol, which will enable integration with Deno.serveHttp/etc and allow WebTransport over h2. This will also let us expose multiplexing.
WebTransport stats will be a followup due to their complexity.
Fixes: #9017