Rack status code name changes breaks HttpStatusMatcher about rspec-rails

[ioquatix]

For what it's worth, Rack just follows the specifications as defined by various RFCs.

Relevant RFCs

Both of the following RFCs define 422 but with a one-word difference:

• RFC 4918, "Proposed Standard", June 2007, "Unprocessable Entity"
• RFC 9110, "Internet Standard", June 2022, "Unprocessable Content"

Potential Confusion: Even though RFC 9110 it is newer, it does not obsolete RFC 4918 (directly nor transitively).

Arguments in Favor of RFC 9110

1. RFC 9110 is titled "HTTP Semantics" which is a direct fit for this library, whereas RFC 4918 is scoped to WebDAV.
2. RFC 9110 is an Internet Standard while RFC 4918 is only a Proposed Standard.
3. MDN uses RFC 9110 for the 422 status code
4. RFC 9110 is newer (2022 vs. 2007)

RFC 9110: 422: Unprocessable Content

15.5.21. 422 Unprocessable Content
The 422 (Unprocessable Content) status code indicates that the server understands the content type of the request content (hence a 415 (Unsupported Media Type) status code is inappropriate), and the syntax of the request content is correct, but it was unable to process the contained instructions. For example, this status code can be sent if an XML request content contains well-formed (i.e., syntactically correct), but semantically erroneous XML instructions.

References

• Update status code 422 for matching rfc9110 mdn/content#24866
• https://httpwg.org/specs/rfc9110.html#status.422
• https://stackoverflow.com/a/52098667

From hyperium/http#664.

[darrenboyd]

Nice catch. Would you like to submit a PR?

The simplest possible PR is here:

#2765

@sarvesh-sankaranarayanan, yeah, I tend to prefer encouraging people to move forward with changes like this. However, this change is more about using Rack's API in a better way. It looks like the Rack developers are going to add a deprecation warning for this change, and then force people to move over a future version.

[ioquatix]

Nice work team!

[pirj]

Nice catch. Would you like to submit a PR?

[sarvesh-sankaranarayanan]

@pirj @darrenboyd Shouldn't we actually ask developers to use the actual changed status code name as per rack-utils, instead of supporting the old name?

[pirj]

This is how the public rack api behaves.

Who knows, might be they’ll decide to change the status back?

How would you interpret the definition of the “symbol to status code” as referenced from the assert_response mapping description in Rails guides?