minor #35090 Add note about HTTP status code change (yceruto)

yceruto · yceruto · commit 0bd742ac0727 · 2019-12-23T11:11:59.000-05:00
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
diff --git a/UPGRADE-4.4.md b/UPGRADE-4.4.md
@@ -306,7 +306,7 @@ TwigBundle
    The new default exception controller will also change the error response content according to
    https://tools.ietf.org/html/rfc7807 for `json`, `xml`, `atom` and `txt` formats:
 
-   Before:
+   Before (HTTP status code `200`):
    ```json
    {
        "error": {
@@ -316,7 +316,7 @@ TwigBundle
    }
    ```
 
-   After:
+   After (HTTP status code `404`):
    ```json
    {
        "title": "Not Found",

Original file line number	Diff line number	Diff line change
`@@ -306,7 +306,7 @@ TwigBundle`
`306`	`306`	`The new default exception controller will also change the error response content according to`
`307`	`307`	https://tools.ietf.org/html/rfc7807 for `json`, `xml`, `atom` and `txt` formats:
`308`	`308`
`309`		`- Before:`
	`309`	+ Before (HTTP status code `200`):
`310`	`310`	```json
`311`	`311`	`{`
`312`	`312`	`"error": {`
`@@ -316,7 +316,7 @@ TwigBundle`
`316`	`316`	`}`
`317`	`317`	```
`318`	`318`
`319`		`- After:`
	`319`	+ After (HTTP status code `404`):
`320`	`320`	```json
`321`	`321`	`{`
`322`	`322`	`"title": "Not Found",`