[HttpFoundation] Fixed BinaryFileResponse sending wrong Content-Length header for files modified by stream wrappers/filters.

TravisCarden · TravisCarden · commit 53f807a32b3d · 2016-08-25T15:51:52.000-05:00
diff --git a/src/Symfony/Component/HttpFoundation/BinaryFileResponse.php b/src/Symfony/Component/HttpFoundation/BinaryFileResponse.php
@@ -186,7 +186,12 @@ public function setContentDisposition($disposition, $filename = '', $filenameFal
      */
     public function prepare(Request $request)
     {
-        $this->headers->set('Content-Length', $this->file->getSize());
+        // Get the file size from a stream rather than from disk in case it is changed later by
+        // stream wrappers or stream filters.
+        $file = fopen($this->file->getPathname(), 'rb');
+        $fstat = fstat($file);
+        $this->headers->set('Content-Length', $fstat['size']);
+        fclose($file);
 
         if (!$this->headers->has('Accept-Ranges')) {
             // Only accept ranges on safe HTTP methods
