fix(http): Set content-length only if body is present

Author: vicb

.changeset/giant-days-develop.md

@@ -0,0 +1,7 @@
+---
+"@opennextjs/aws": patch
+---
+
+fix(http): Set content-length only if body is present
+
+The body is undefined when using the edge converter and the method is GET or HEAD

packages/open-next/src/http/request.ts

@@ -6,19 +6,15 @@
 import http from "node:http";
 
 export class IncomingMessage extends http.IncomingMessage {
-  constructor({
-    method,
-    url,
-    headers,
-    body,
-    remoteAddress,
-  }: {
+  constructor(request: {
     method: string;
     url: string;
     headers: Record<string, string | string[]>;
     body?: Buffer;
     remoteAddress?: string;
   }) {
+    const { method, url, headers, body, remoteAddress } = request;
+
     super({
       encrypted: true,
       readable: false,
@@ -28,8 +24,10 @@ export class IncomingMessage extends http.IncomingMessage {
       destroy: Function.prototype,
     });
 
-    if (typeof headers["content-length"] === "undefined") {
-      headers["content-length"] = Buffer.byteLength(body).toString();
+    // Set the content length when there is a body.
+    // See https://httpwg.org/specs/rfc9110.html#field.content-length
+    if (body) {
+      headers["content-length"] ??= String(Buffer.byteLength(body));
     }
 
     Object.assign(this, {