-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Use upstream websocket crate. #16236
Conversation
Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @asajeffrey (or someone else) soon. |
Heads up! This PR modifies the following files:
|
CC @avadacatavra, @nox |
That's exactly why,
Because in the current setup, we block on reading in a thread, and we write on a different thread, so the
No, we only do a subset of what the |
Note that I actually tried to update to last |
@nox so as I understand it you would like to see:
As far as I know no. 3 doesn't have a good answer yet, since |
Yep! |
@illegalprime I forgot to say it earlier, but thanks for looking at this btw! |
I've merged |
Closing due to inactivity. |
Recently I've finished another release of the
websocket
crate, so there's lots of neat stuff to use. I wanted to see how other projects used this library, and I thought this would be a great place to check.I have some questions on the implementation here that will inform the future design of the
websocket
crate:Mutex<>
or something?fetch
functions just check and set headers, is it worth keeping the structure or just building a list of headers and setting them in the client builder?NOTE: this is not ready to be merged, there are some
.unwrap()
calls and it doesn't set any of the important headers listed later in the file.ANOTHER NOTE: this websockets release is going to go through some more testing before being published, and is also blocked by hyperium/hyper#1107 being included in a hyper version.
Thanks!
./mach build -d
does not report any errors./mach test-tidy
does not report any errorsThis change is