Content-Transfer-Encoding value not case sensitive

ThaDafinser · Mar 25, 2014 · abbbeea · abbbeea
1 parent 606f6fe
commit abbbeea
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/library/Zend/Mail/Header/ContentTransferEncoding.php b/library/Zend/Mail/Header/ContentTransferEncoding.php
@@ -91,6 +91,9 @@ public function toString()
      */
     public function setTransferEncoding($transferEncoding)
     {
+        // Per RFC 1521, the value of the header is not case sensitive
+        $transferEncoding = strtolower($transferEncoding);
+
         if (!in_array($transferEncoding, self::$allowedTransferEncodings)) {
             throw new Exception\InvalidArgumentException(sprintf(
                 '%s expects one of "'. implode(', ', self::$allowedTransferEncodings) . '"; received "%s"',

diff --git a/tests/ZendTest/Mail/Header/ContentTransferEncodingTest.php b/tests/ZendTest/Mail/Header/ContentTransferEncodingTest.php
@@ -43,6 +43,13 @@ public function testContentTransferEncodingGetFieldValueReturnsProperValue()
         $this->assertEquals('7bit', $contentTransferEncodingHeader->getFieldValue());
     }
 
+    public function testContentTransferEncodingHandlesCaseInsensitivity()
+    {
+        $encoding = new ContentTransferEncoding();
+        $encoding->setTransferEncoding('quOtED-printAble');
+        $this->assertEquals('quoted-printable', strtolower($encoding->getFieldValue()));
+    }
+
     public function testContentTransferEncodingToStringReturnsHeaderFormattedString()
     {
         $contentTransferEncodingHeader = new ContentTransferEncoding();