Fixed the zero value in ContentLength

zendframework · Aug 8, 2016 · b5cc5ba · b5cc5ba
1 parent e9fb717
commit b5cc5ba
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/Header/ContentLength.php b/src/Header/ContentLength.php
@@ -40,7 +40,7 @@ public static function fromString($headerLine)
 
     public function __construct($value = null)
     {
-        if ($value) {
+        if (null !== $value) {
             HeaderValue::assertValid($value);
             $this->value = $value;
         }

diff --git a/test/Header/ContentLengthTest.php b/test/Header/ContentLengthTest.php
@@ -65,4 +65,11 @@ public function testPreventsCRLFAttackViaConstructor()
         $this->setExpectedException('Zend\Http\Header\Exception\InvalidArgumentException');
         $header = new ContentLength("Content-Length: xxx\r\n\r\nevilContent");
     }
+
+    public function testZeroValue()
+    {
+        $contentLengthHeader = new ContentLength(0);
+        $this->assertEquals(0, $contentLengthHeader->getFieldValue());
+        $this->assertEquals('Content-Length: 0', $contentLengthHeader->toString());
+    }
 }