Subprotocol header #363
Subprotocol header #363
Conversation
|
@daniel-abramov Are you interested in merging this? |
|
Hey, I think we've forgotten about this one, sorry (though the code looks familiar as if I have already skewed through it once). Generally, it looks ok - there is an intersection with #400 though, we'll need to check how to merge it without causing too many conflicts. UPD: I'd also need to check why no CI pipeline was run for this PR. |
|
Thanks for getting back to me. I'll update it once #400 is progressed. |
|
Hello @conorbros Here is a PR to merge shotover#1 to sync with the repo |
Change tests to merge snapview#363
There was a problem hiding this comment.
Hi @conorbros
Can you please juste made this tiny changes to passe the CI ?
FYI CI error is here : https://github.com/snapview/tungstenite-rs/actions/runs/8841412949/job/24741691622
Co-authored-by: n4n5 <56606507+Its-Just-Nans@users.noreply.github.com>
|
Ok, now it looks good. All tests are passed, the new code is covered with tests as well. Its time to merge. |
Add client-side handling for subprotocol header during handshake.