[HttpFoundation] Set the Content-Range header if the requested Range is unsatisfied

jakzal · jakzal · commit 54329d836c0a · 2016-03-09T16:57:57.000Z
diff --git a/src/Symfony/Component/HttpFoundation/BinaryFileResponse.php b/src/Symfony/Component/HttpFoundation/BinaryFileResponse.php
@@ -252,6 +252,7 @@ public function prepare(Request $request)
                 if ($start <= $end) {
                     if ($start < 0 || $end > $fileSize - 1) {
                         $this->setStatusCode(416);
+                        $this->headers->set('Content-Range', sprintf('bytes */%s', $fileSize));
                     } elseif ($start !== 0 || $end !== $fileSize - 1) {
                         $this->maxlen = $end < $fileSize ? $end - $start + 1 : -1;
                         $this->offset = $start;
diff --git a/src/Symfony/Component/HttpFoundation/Tests/BinaryFileResponseTest.php b/src/Symfony/Component/HttpFoundation/Tests/BinaryFileResponseTest.php
@@ -211,7 +211,7 @@ public function testInvalidRequests($requestRange)
         $response->sendContent();
 
         $this->assertEquals(416, $response->getStatusCode());
-        #$this->assertEquals('', $response->headers->get('Content-Range'));
+        $this->assertEquals('bytes */35', $response->headers->get('Content-Range'));
     }
 
     public function provideInvalidRanges()

Original file line number	Diff line number	Diff line change
`@@ -211,7 +211,7 @@ public function testInvalidRequests($requestRange)`
`211`	`211`	`$response->sendContent();`
`212`	`212`
`213`	`213`	`$this->assertEquals(416, $response->getStatusCode());`
`214`		`- #$this->assertEquals('', $response->headers->get('Content-Range'));`
	`214`	`+ $this->assertEquals('bytes */35', $response->headers->get('Content-Range'));`
`215`	`215`	`}`
`216`	`216`
`217`	`217`	`public function provideInvalidRanges()`