-
Notifications
You must be signed in to change notification settings - Fork 567
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
reason phrase #202
Comments
Discussed in Seattle; no interest in carrying the reason phrase. |
i'm curious what was the logical reason for dropping the reason phrase? i was using the reason phrase as a title for messages presented to a user in the web browser client. i think most users are accustomed to such phrases, "Bad Request", "Not Found", etc. Now I will just have to write a mapping from status codes to my own reason phrases in the client. |
The reason phrase -- even in HTTP/1.1 -- isn't guaranteed to be carried end-to-end; implementations can (and do) ignore it and substitute their own values (e.g., 200 is always "OK", no matter what happens on the wire). Given that, and overhead in carrying the extra bytes, it made sense to drop it from the wire. |
It's always been like this, but the documentation never said so. See httpwg/http2-spec#202 for why. Task-number: QTBUG-126278 Pick-to: 6.8 6.7 6.5 Change-Id: Ie83b105dd78fc1c6c09db46ae8cb5e113cd8a062 Reviewed-by: Mate Barany <mate.barany@qt.io>
It's always been like this, but the documentation never said so. See httpwg/http2-spec#202 for why. Task-number: QTBUG-126278 Pick-to: 6.7 6.5 Change-Id: Ie83b105dd78fc1c6c09db46ae8cb5e113cd8a062 Reviewed-by: Mate Barany <mate.barany@qt.io> (cherry picked from commit 365a1ba) Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
It's always been like this, but the documentation never said so. See httpwg/http2-spec#202 for why. Task-number: QTBUG-126278 Pick-to: 6.5 Change-Id: Ie83b105dd78fc1c6c09db46ae8cb5e113cd8a062 Reviewed-by: Mate Barany <mate.barany@qt.io> (cherry picked from commit 365a1ba) Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> (cherry picked from commit fd37035)
8.1.2.:
"The response status line has been reduced to a single ":status" header field whose value specifies only the numeric response status code. The status text component of the HTTP/1.1 response has been dropped entirely."
Dropping the reason-phrase might break a few edge cases; is this worth it?
Maybe add a separate optional ":reason-phrase" header field?
The text was updated successfully, but these errors were encountered: