Skip to content

Commit

Permalink
src: bring 425 status code name into accordance with RFC 8470
Browse files Browse the repository at this point in the history
PR-URL: nodejs#29880
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
hcz authored and Trott committed Oct 10, 2019
1 parent 8b9a578 commit 458a38c
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/_http_server.js
Expand Up @@ -112,7 +112,7 @@ const STATUS_CODES = {
422: 'Unprocessable Entity', // RFC 4918
423: 'Locked', // RFC 4918
424: 'Failed Dependency', // RFC 4918
425: 'Unordered Collection', // RFC 4918
425: 'Too Early', // RFC 8470
426: 'Upgrade Required', // RFC 2817
428: 'Precondition Required', // RFC 6585
429: 'Too Many Requests', // RFC 6585
Expand Down
2 changes: 1 addition & 1 deletion src/node_http2.h
Expand Up @@ -287,7 +287,7 @@ enum http_known_headers {
V(UNPROCESSABLE_ENTITY, 422) \
V(LOCKED, 423) \
V(FAILED_DEPENDENCY, 424) \
V(UNORDERED_COLLECTION, 425) \
V(TOO_EARLY, 425) \
V(UPGRADE_REQUIRED, 426) \
V(PRECONDITION_REQUIRED, 428) \
V(TOO_MANY_REQUESTS, 429) \
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-http2-binding.js
Expand Up @@ -74,7 +74,7 @@ const expectedStatusCodes = {
HTTP_STATUS_UNPROCESSABLE_ENTITY: 422,
HTTP_STATUS_LOCKED: 423,
HTTP_STATUS_FAILED_DEPENDENCY: 424,
HTTP_STATUS_UNORDERED_COLLECTION: 425,
HTTP_STATUS_TOO_EARLY: 425,
HTTP_STATUS_UPGRADE_REQUIRED: 426,
HTTP_STATUS_PRECONDITION_REQUIRED: 428,
HTTP_STATUS_TOO_MANY_REQUESTS: 429,
Expand Down

0 comments on commit 458a38c

Please sign in to comment.