Change range header to be inclusive

Phalcode · Jun 16, 2024 · 5d351f4 · 5d351f4
1 parent ede6ef4
commit 5d351f4
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/modules/files/files.service.ts b/src/modules/files/files.service.ts
@@ -675,7 +675,7 @@ export class FilesService implements OnApplicationBootstrap {
     fileSize: number,
   ): RangeHeader {
     let rangeStart: number = 0;
-    let rangeEnd: number = fileSize;
+    let rangeEnd: number = fileSize - 1;
 
     if (rangeHeader?.includes("-")) {
       const [extractedStart, extractedEnd] = rangeHeader
@@ -688,14 +688,14 @@ export class FilesService implements OnApplicationBootstrap {
       }
       if (
         !isNaN(extractedEnd) &&
-        extractedEnd > rangeStart &&
+        extractedEnd >= rangeStart &&
         extractedEnd < fileSize
       ) {
         rangeEnd = extractedEnd;
       }
     }
 
-    const rangeSize: number = rangeEnd - rangeStart;
+    const rangeSize: number = rangeEnd - rangeStart + 1;
     return {
       start: rangeStart,
       end: rangeEnd,