fix(Core): fixes limit stream errors (#7117)

src/Upload/ResumableUploader.php

@@ -274,12 +274,13 @@ protected function getStatusResponse()
      * Gets the starting range for the upload.
      *
      * @param string $rangeHeader
-     * @return int|null
+     * @return int
      */
     protected function getRangeStart($rangeHeader)
     {
         if (!$rangeHeader) {
-            return null;
+            // assume no bytes are uploaded if no range header is present
+            return 0;
         }
 
         return (int) explode('-', $rangeHeader)[1] + 1;

tests/Unit/Upload/ResumableUploaderTest.php

@@ -293,4 +293,31 @@ public function testThrowsExceptionWithFailedUpload()
 
         $uploader->upload();
     }
+
+    /**
+     * @dataProvider rangeHeaderProvider
+     */
+    public function testGetRangeStart($rangeHeader, $expectedRangeStart)
+    {
+        $method = new \ReflectionMethod(ResumableUploader::class, 'getRangeStart');
+        $method->setAccessible(true);
+
+        $uploader = $this->createMock(ResumableUploader::class);
+
+        $actualRangeStart = $method->invoke($uploader, $rangeHeader);
+        $this->assertEquals($expectedRangeStart, $actualRangeStart);
+    }
+
+    public function rangeHeaderProvider()
+    {
+        return
+        [
+            // range header, expected range start
+            ['', 0],
+            ['bytes 0-3/4', 4],
+            ['bytes 0-99/100', 100],
+            ['bytes 100-199/200', 200],
+            ['bytes=2000-',1]
+        ];
+    }
 }