WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 6 weeks ago

#16914 closed enhancement (fixed)

Updated List of HTTP status codes for wp_header_to_desc()

Reported by: hakre Owned by: johnbillion
Milestone: 3.9 Priority: low
Severity: minor Version: 3.1
Component: HTTP API Keywords: has-patch commit
Focuses: Cc:

Description

http://www.iana.org/assignments/http-status-codes

Hypertext Transfer Protocol (HTTP) Status Code Registry

(last updated 2010-11-05)

Related: #9297

Attachments (4)

16914.patch (910 bytes) - added by hakre 3 years ago.
Update to latest version of the IANA document.
16914.2.patch (4.1 KB) - added by hakre 3 years ago.
Update to latest version of the IANA document. (with comments)
19601.diff (5.1 KB) - added by sirzooro 2 years ago.
Patch attached to #19596, with comments added
16914.3.patch (905 bytes) - added by johnbillion 6 weeks ago.

Download all attachments as: .zip

Change History (13)

comment:1 hakre3 years ago

  • Keywords has-patch added
  • Type changed from defect to enhancement
  • Version set to 3.1

hakre3 years ago

Update to latest version of the IANA document.

hakre3 years ago

Update to latest version of the IANA document. (with comments)

comment:2 hakre3 years ago

Those two patches contain the same additions to the list (as of the current IANA document), while the second patch contains more meta information in comments.

Last edited 3 years ago by hakre (previous) (diff)

comment:3 ocean902 years ago

Duplicate: #19596

sirzooro2 years ago

Patch attached to #19596, with comments added

comment:4 sirzooro2 years ago

  • Cc sirzooro added

comment:5 nacin3 months ago

  • Component changed from General to HTTP
  • Milestone changed from Awaiting Review to 3.9

comment:6 ircbot7 weeks ago

This ticket was mentioned in IRC in #wordpress-dev by nacin. View the logs.

comment:7 johnbillion6 weeks ago

  • Keywords needs-refresh added; has-patch removed
  • Priority changed from normal to low
  • Severity changed from normal to minor

19601.diff is a bit too liberal in its implementation. From that patch:

Status codes that should be added:

  • 428, 429, 431, and 511 are part of RFC 6585 which is still a proposal, but everyone seems to have implemented it anyway, including PHP 5.5. Safe to go in.
  • 418 is an April Fools' joke, but I'm sure it's safe enough to go in.

Status codes that should not be added:

  • 122 and 450 are not actual HTTP response codes. They occur internally in IE7 and Windows respectively.
  • 103 and 308 are from an outdated proposal. 308 is now part of an experimental draft.
  • 425 remains unassigned and was only ever pre-allocated for RFC 2817.
  • 444 and 499 are, by their definitions, not actual status codes and are only used internally by Nginx.
  • 509 is unassigned. Despite this code and its description being included on some "HTTP status code" websites, its origin and use is unknown.
  • 510 is part of RFC 2774 which isn't implemented anywhere.

johnbillion6 weeks ago

comment:8 johnbillion6 weeks ago

  • Keywords has-patch commit added; needs-refresh removed
  • Summary changed from Updated List of HTTP status codes for wp_header_to_desc (2) to Updated List of HTTP status codes for wp_header_to_desc()

comment:9 johnbillion6 weeks ago

  • Owner set to johnbillion
  • Resolution set to fixed
  • Status changed from new to closed

In 27422:

Update the list of HTTP status code descriptions. Fixes #16914. Props sirzooro and hakre for the original patches.

Note: See TracTickets for help on using tickets.