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

HTTP Upgrade in H3 is silliness #2898

Merged
merged 1 commit into from
Aug 28, 2019
Merged

Conversation

LPardue
Copy link
Member

@LPardue LPardue commented Jul 15, 2019

Fixes #2889

@ianswett ianswett added the -http label Jul 15, 2019
@MikeBishop
Copy link
Contributor

Is it worth adding a reference to RFC 8441? That's an alternative that covers what people looking for this functionality were probably trying to do.

@LPardue
Copy link
Member Author

LPardue commented Aug 6, 2019

That's a good point, let me craft something up and I'll update the PR soon.

@MikeBishop
Copy link
Contributor

Ping @LPardue, unless you'd like me to take it over.

@LPardue
Copy link
Member Author

LPardue commented Aug 27, 2019

I'll get an update out tomorrow

@LPardue
Copy link
Member Author

LPardue commented Aug 28, 2019

I'm actually not convinced that referencing 8441 will help clarify things here because the extension is H2 specific. No H3-equivalent exists (or at least, no H3 setting to enable the equivalent CONNECT behaviour exists).

@bagder
Copy link

bagder commented Aug 28, 2019

I agree. I think referencing 8441 might just add to the confusion - even if something like 8441 certainly can be introduced.

@LPardue
Copy link
Member Author

LPardue commented Aug 28, 2019

Some others have shown interest for defining/framing this capability in HTTP/3. I think that activity should consider ore broadly what text makes sense in the core document.

@MikeBishop
Copy link
Contributor

Fair enough. Let's merge the rebased version, then, and leave the right way to do this for an H3 extension.

@MikeBishop MikeBishop merged commit 1f913fd into quicwg:master Aug 28, 2019
@juliens juliens mentioned this pull request Oct 12, 2019
28 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HTTP/3 support for status 101(Switching Protocols)
4 participants