Make http.Response.Status respect RFC 2616, like in Go stdlib.#159
Merged
Conversation
maxatome
reviewed
Nov 14, 2024
Collaborator
|
Thanks @constantoine, could you squash your commits please? |
Although RFC 2616 has been superseded with RFC 9110, some client implementation still rely on certain behavious regarding the Status-Line. Now, as httpmock currently implements it, per [RFC 9110#6.2-1](https://www.rfc-editor.org/rfc/rfc9110.html#section-6.2-1), the "reason phrase" is actually optional, and you need only to include the status code. Back in the days of [RFC 2616](https://www.rfc-editor.org/rfc/rfc2616.html#section-6.1), per jarcoal#6.1, the Status-Line had to contain both the status code (eg. 200) and its textual representation (eg. OK). Change code is directly stolen from the Go standard library. Signed-off-by: Cléo Rebert <cleo.rebert@gmail.com>
Contributor
Author
|
@maxatome I think we should be good :) |
Collaborator
|
Thanks @constantoine! I hope to publish a new release soon, I work on a new feature not fully ready yet. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Although RFC 2616 has been superseded with RFC 9110, some client implementation still rely on certain behavious regarding the Status-Line.
Now, as httpmock currently implements it, per RFC 9110#6.2-1, the "reason phrase" is actually optional, and you need only to include the status code.
Back in the days of RFC 2616#6.1, the Status-Line had to contain both the status code (eg. 200) and its textual representation (eg. OK).
Change code is directly stolen from the Go standard library.
This aims to fix #83 .