feat: Remove default content-length header from MultipartStreamBuilder class

Author: chris-lee-lb

src/MultipartStreamBuilder.php

@@ -185,13 +185,6 @@ private function prepareHeaders($name, StreamInterface $stream, $filename, array
             }
         }
 
-        // Set a default content-length header if one was not provided
-        if (!$this->hasHeader($headers, 'content-length')) {
-            if ($length = $stream->getSize()) {
-                $headers['Content-Length'] = (string) $length;
-            }
-        }
-
         // Set a default Content-Type if one was not provided
         if (!$this->hasHeader($headers, 'content-type') && $hasFilename) {
             if ($type = $this->getMimetypeHelper()->getMimetypeFromFilename($filename)) {

tests/FunctionTest.php

@@ -88,13 +88,13 @@ public function testHeaders()
         $this->assertTrue(false === strpos($multipartStream, 'Content-Disposition:'));
     }
 
-    public function testContentLength()
+    public function testShouldNotContainContentLength()
     {
         $builder = new MultipartStreamBuilder();
         $builder->addResource('foobar', 'stream contents');
 
         $multipartStream = (string) $builder->build();
-        $this->assertTrue(false !== strpos($multipartStream, 'Content-Length: 15'));
+        $this->assertTrue(false === strpos($multipartStream, 'Content-Length: 15'));
     }
 
     public function testFormName()