Changeset 59899 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-search-controller.php

Timestamp:

03/02/2025 10:05:08 PM (3 months ago)

Author:

TimothyBlynJacobs

Message:

REST API: Improve performance for HEAD requests.

By default, the REST API responds to HEAD rqeuests by calling the GET handler and omitting the body from the response. While convenient, this ends up performing needless work that slows down the API response time.

This commit adjusts the Core controllers to specifically handle HEAD requests by not preparing the response body.

Fixes #56481.
Props antonvlasenko, janusdev, ironprogrammer, swissspidy, spacedmonkey, mukesh27, mamaduka, timothyblynjacobs.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-search-controller.php (modified) (2 diffs)

trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-search-controller.php

@@ -143 +143 @@
         $ids = $result[ WP_REST_Search_Handler::RESULT_IDS ];
 
-        $results = array();
-
-        foreach ( $ids as $id ) {
-            $data      = $this->prepare_item_for_response( $id, $request );
-            $results[] = $this->prepare_response_for_collection( $data );
+        $is_head_request = $request->is_method( 'HEAD' );
+        if ( ! $is_head_request ) {
+            $results = array();
+
+            foreach ( $ids as $id ) {
+                $data      = $this->prepare_item_for_response( $id, $request );
+                $results[] = $this->prepare_response_for_collection( $data );
+            }
         }
 
@@ -163 +166 @@
         }
 
-        $response = rest_ensure_response( $results );
+        $response = $is_head_request ? new WP_REST_Response() : rest_ensure_response( $results );
         $response->header( 'X-WP-Total', $total );
         $response->header( 'X-WP-TotalPages', $max_pages );