WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 3 years 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 6 years ago.
Update to latest version of the IANA document.
16914.2.patch (4.1 KB) - added by hakre 6 years ago.
Update to latest version of the IANA document. (with comments)
19601.diff (5.1 KB) - added by sirzooro 5 years ago.
Patch attached to #19596, with comments added
16914.3.patch (905 bytes) - added by johnbillion 3 years ago.

Download all attachments as: .zip

Change History (13)

#1 @hakre
6 years ago

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

@hakre
6 years ago

Update to latest version of the IANA document.

@hakre
6 years ago

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

#2 @hakre
6 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 6 years ago by hakre (previous) (diff)

#3 @ocean90
5 years ago

Duplicate: #19596

@sirzooro
5 years ago

Patch attached to #19596, with comments added

#4 @sirzooro
5 years ago

  • Cc sirzooro added

#5 @nacin
3 years ago

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

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


3 years ago

#7 @johnbillion
3 years 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.

@johnbillion
3 years ago

#8 @johnbillion
3 years 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()

#9 @johnbillion
3 years 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.