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
Rewrite proxy-to-origin coalescing case #99
Comments
|
This issue has probably several linked parts:
I think this is now covered by #110
We discussed not specifying exactly and #110 gives examples. Authentication, session information etc.
|
|
I don't agree we discussed not to have a separate signal. The text in the HTTP/1.1 case is good and as I understood it, but I think we need something explicit for H2 and H3 when coalescing multiple clients. cc @MikeBishop. If you don't specify this for H2 and H3, there's imo a low chance origin servers will actually do anything intelligently in this case. |
|
Sorry, I used a contraction: A separate signal is required, but I think we agreed not to specify that signal and just to highlight that one was required. |
|
In that case, I would like us to revisit that agreement :) (not necessarily for the current version of course) |
|
I do not think we need to resolve this in the next draft. Therefore added a TODO to the I-D, referencing this issue from there. |
Discussed in Singapore.
Common modus operandi should be that proxies generally keep the client-specified priorities intact when forwarding. If they do, they should also set a separate signal to let the origin know it's behind a proxy that's coalescing multiple clients on 1 connection. We should also specify this signal (related to #57 but different, cc @MikeBishop). We then also need text specifying how the origin should respond to that signal (i.e., we can get rid of #77, cc @kazuho).
Should also discuss the case that if proxies want to change the priority, they should use a PRIORITY_UPDATE frame immediately after the HEADERS (solves the "how do we communicate original priorities to the origin" issue).
The text was updated successfully, but these errors were encountered: