fix(openapi-fetch): prevent body parsing for HEAD requests regardless of Content-Length (#2205)

* fix(openapi-fetch): prevent body parsing for HEAD requests regardless of Content-Length

* chore: add changeset for HEAD method fix

Author: live-small

.changeset/breezy-bottles-mix.md

@@ -0,0 +1,5 @@
+---
+"openapi-fetch": patch
+---
+
+Fix HEAD method requests to prevent body parsing regardless of Content-Length header value

packages/openapi-fetch/src/index.js

@@ -209,7 +209,7 @@ export default function createClient(clientOptions) {
     }
 
     // handle empty content
-    if (response.status === 204 || response.headers.get("Content-Length") === "0") {
+    if (response.status === 204 || request.method === "HEAD" || response.headers.get("Content-Length") === "0") {
       return response.ok ? { data: undefined, response } : { error: undefined, response };
     }
 

packages/openapi-fetch/test/http-methods/head.test.ts

@@ -13,4 +13,16 @@ describe("HEAD", () => {
     await client.HEAD("/resources/{id}", { params: { path: { id: 123 } } });
     expect(method).toBe("HEAD");
   });
+
+  test("handles HEAD requests with non-zero Content-Length without parsing the body", async () => {
+    const client = createObservedClient<paths>({}, async () => {
+      return new Response(null, {
+        headers: { "Content-Length": "42", "Content-Type": "application/json" },
+        status: 200,
+      });
+    });
+    const result = await client.HEAD("/resources/{id}", { params: { path: { id: 123 } } });
+    expect(result.data).toBeUndefined();
+    expect(result.response.ok).toBe(true);
+  });
 });