fix(Response): single Content-Range in http response

src/bun.js/api/server.zig

@@ -3008,9 +3008,11 @@ fn NewRequestContext(comptime ssl_enabled: bool, comptime debug_mode: bool, comp
             };
 
             var has_content_disposition = false;
+            var has_content_range = false;
             if (response.init.headers) |headers_| {
                 has_content_disposition = headers_.fastHas(.ContentDisposition);
-                needs_content_range = needs_content_range and headers_.fastHas(.ContentRange);
+                has_content_range = headers_.fastHas(.ContentRange);
+                needs_content_range = needs_content_range and has_content_range;
                 if (needs_content_range) {
                     status = 206;
                 }
@@ -3057,7 +3059,7 @@ fn NewRequestContext(comptime ssl_enabled: bool, comptime debug_mode: bool, comp
                 this.flags.needs_content_length = false;
             }
 
-            if (needs_content_range) {
+            if (needs_content_range and !has_content_range) {
                 var content_range_buf: [1024]u8 = undefined;
 
                 resp.writeHeader(

test/js/bun/http/serve.test.ts

@@ -226,7 +226,7 @@
             return new Response(
               new ReadableStream({
                 pull(controller) {
                   throw new Error("TestPassed");
                 },
                 cancel(reason) {},
               }),
@@ -263,7 +263,7 @@
                 async pull(controller) {
                   controller.enqueue("PASS");
                   controller.close();
                   throw new Error("FAIL");
                 },
               });
               return new Response(stream, options);
@@ -947,6 +947,18 @@
     },
   });
 
+  const getServerWithSize = runTest.bind(null, {
+    fetch(req) {
+      const { searchParams } = new URL(req.url);
+      const start = Number(searchParams.get("start"));
+      const end = Number(searchParams.get("end"));
+      const file = Bun.file(fixture);
+      return new Response(file.slice(start, end), {
+        headers: { "Content-Range": "bytes " + start + "-" + end + "/" + file.size },
+      });
+    },
+  });
+
   const good = [
     [0, 1],
     [1, 2],
@@ -973,6 +985,19 @@
     });
   }
 
+  for (const [start, end] of good) {
+    it(`good range with size: ${start} - ${end}`, async () => {
+      await getServerWithSize(async server => {
+        const response = await fetch(`http://${server.hostname}:${server.port}/?start=${start}&end=${end}`, {
+          verbose: true,
+        });
+        expect(parseInt(response.headers.get("Content-Range")?.split("/")[1])).toEqual(full.byteLength);
+        expect(await response.arrayBuffer()).toEqual(full.buffer.slice(start, end));
+        expect(response.status).toBe(start > 0 || end < full.byteLength ? 206 : 200);
+      });
+    });
+  }
+
   const emptyRanges = [
     [0, 0],
     [1, 1],