WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 11 years ago

#16914 closed enhancement (fixed)

Updated List of HTTP status codes for wp_header_to_desc()

Reported by: hakre's profile hakre Owned by: johnbillion's profile 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 14 years ago.
Update to latest version of the IANA document.
16914.2.patch​ (4.1 KB) - added by hakre 14 years ago.
Update to latest version of the IANA document. (with comments)
19601.diff​ (5.1 KB) - added by sirzooro 13 years ago.
Patch attached to #19596, with comments added
16914.3.patch​ (905 bytes) - added by johnbillion 11 years ago.

Download all attachments as: .zip

Change History (13)

#1 @hakre
14 years ago

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

@hakre
14 years ago

Update to latest version of the IANA document.

@hakre
14 years ago

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

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

#3 @ocean90
13 years ago

Duplicate: #19596

@sirzooro
13 years ago

Patch attached to #19596, with comments added

#4 @sirzooro
13 years ago

  • Cc sirzooro added

#5 @nacin
11 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.
11 years ago

#7 @johnbillion
11 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
11 years ago

#8 @johnbillion
11 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
11 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.