WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#34837 closed defect (bug) (fixed)

Fix handling of HEAD requests in WP_REST_Server

Reported by: danielbachhuber Owned by: markjaquith
Priority: normal Milestone: 4.4
Component: REST API Version: 4.4
Severity: normal Keywords: has-patch has-unit-tests commit
Cc: Focuses:

Description

HEAD request handling has been broken in all of the 2.x betas because, surprise, there was no test coverage for it.

The intended behavior for a HEAD request is to use the GET callback method. See how this is ​handled in 1.2.4

public function dispatch() {
        switch ( $this->method ) {
            case 'HEAD':
            case 'GET':
                $method = self::METHOD_GET;
                break;

We should do equivalent in the REST server infrastructure.

Attachments (1)

34837.1.diff​ (1.4 KB ) - added by danielbachhuber 11 years ago.

Download all attachments as: .zip

Change History (5)

@danielbachhuber
11 years ago

#1 @markjaquith
11 years ago

  • Keywords 2nd-opinion added

Looks good. Asking for secondary review.

This ticket was mentioned in ​Slack in #core by mark. ​View the logs.
11 years ago

#3 @helen
11 years ago

  • Keywords commit added; 2nd-opinion removed

Go forth.

#4 @markjaquith
11 years ago

  • Owner set to markjaquith
  • Resolution → fixed
  • Status new → closed

In 35758:

Route HEAD API requests through the GET callback method

fixes #34837
props danielbachhuber

Note: See TracTickets for help on using tickets.