Apply suggestions from code review

Author: dbu

Diff for: CHANGELOG.md

@@ -1,8 +1,8 @@
 # Change Log
 
-## 1.3.2 - 2024-08-10
+## 1.4.0 - 2024-09-01
 
-- Remove default `Content-Length` header from MultipartStreamBuilder class.
+- No longer automatically add a `Content-Length` header for each part in MultipartStreamBuilder class to comply with RFC 7578 section 4.8.
 
 ## 1.3.1 - 2024-06-10
 

Diff for: src/MultipartStreamBuilder.php

@@ -95,9 +95,9 @@ public function addData($resource, array $headers = [])
      * @param string|resource|StreamInterface $resource
      * @param array                           $options  {
      *
-     * @var array  $headers additional headers ['header-name' => 'header-value']
-     * @var string $filename
-     *             }
+     *     @var array  $headers additional headers ['header-name' => 'header-value']
+     *     @var string $filename
+     * }
      *
      * @return MultipartStreamBuilder
      */

Diff for: tests/FunctionTest.php

@@ -88,13 +88,16 @@ public function testHeaders()
         $this->assertTrue(false === strpos($multipartStream, 'Content-Disposition:'));
     }
 
+    /**
+     * Comply with RFC 7578 section 4.8
+     */
     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:'));
     }
 
     public function testFormName()