Skip to content

Add note about HTTP status code change #35090

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 23, 2019
Merged

Add note about HTTP status code change #35090

merged 1 commit into from
Dec 23, 2019

Conversation

yceruto
Copy link
Member

@yceruto yceruto commented Dec 23, 2019 •
edited
Loading

Q A
Branch? 4.4
Bug fix? no
New feature? no
Deprecations? no
Tickets Fix #34731
License MIT
Doc PR -

https://tools.ietf.org/html/rfc7807
The "status" member, if present, is only advisory; it conveys the
HTTP status code used for the convenience of the consumer.
Generators MUST use the same status code in the actual HTTP response,
to assure that generic HTTP software that does not understand this
format still behaves correctly.

@yceruto yceruto added this to the 4.4 milestone Dec 23, 2019
yceruto added a commit that referenced this pull request Dec 23, 2019
@yceruto
minor #35090 Add note about HTTP status code change (yceruto)
0bd742a 
This PR was merged into the 4.4 branch.

Discussion
----------

Add note about HTTP status code change

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #34731
| License       | MIT
| Doc PR        | -

> https://tools.ietf.org/html/rfc7807
> The "status" member, if present, is only advisory; it conveys the
   HTTP status code used for the convenience of the consumer.
   **Generators MUST use the same status code in the actual HTTP response,**
   to assure that generic HTTP software that does not understand this
   format still behaves correctly.

Commits
-------

429605b add note about HTTP status code change
@yceruto yceruto merged commit 429605b into symfony:4.4 Dec 23, 2019
@yceruto yceruto deleted the error_status_code branch December 23, 2019 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chalasr chalasr chalasr approved these changes

Assignees
No one assigned
Labels
Status: Reviewed TwigBundle
Projects
None yet
Milestone
4.4
Development

Successfully merging this pull request may close these issues.
3 participants
@yceruto @chalasr @carsonbot