[Mime] Fix case-sensitive handling in Headers::isUniqueHeader()

piku235 · piku235 · commit b2d7454042c5 · 2021-02-01T19:58:28.000+01:00
diff --git a/src/Symfony/Component/Mime/Header/Headers.php b/src/Symfony/Component/Mime/Header/Headers.php
@@ -201,7 +201,7 @@ public function remove(string $name): void
 
     public static function isUniqueHeader(string $name): bool
     {
-        return \in_array($name, self::UNIQUE_HEADERS, true);
+        return \in_array(strtolower($name), self::UNIQUE_HEADERS, true);
     }
 
     public function toString(): string
diff --git a/src/Symfony/Component/Mime/Tests/Header/HeadersTest.php b/src/Symfony/Component/Mime/Tests/Header/HeadersTest.php
@@ -212,6 +212,11 @@ public function testRemoveIsNotCaseSensitive()
         $this->assertFalse($headers->has('Message-ID'));
     }
 
+    public function testIsUniqueHeaderIsNotCaseSensitive()
+    {
+        $this->assertTrue(Headers::isUniqueHeader('From'));
+    }
+
     public function testToStringJoinsHeadersTogether()
     {
         $headers = new Headers();

Original file line number	Diff line number	Diff line change
`@@ -201,7 +201,7 @@ public function remove(string $name): void`
`201`	`201`
`202`	`202`	`public static function isUniqueHeader(string $name): bool`
`203`	`203`	`{`
`204`		`- return \in_array($name, self::UNIQUE_HEADERS, true);`
	`204`	`+ return \in_array(strtolower($name), self::UNIQUE_HEADERS, true);`
`205`	`205`	`}`
`206`	`206`
`207`	`207`	`public function toString(): string`
Original file line number	Diff line number	Diff line change
`@@ -212,6 +212,11 @@ public function testRemoveIsNotCaseSensitive()`
`212`	`212`	`$this->assertFalse($headers->has('Message-ID'));`
`213`	`213`	`}`
`214`	`214`
	`215`	`+ public function testIsUniqueHeaderIsNotCaseSensitive()`
	`216`	`+ {`
	`217`	`+ $this->assertTrue(Headers::isUniqueHeader('From'));`
	`218`	`+ }`
	`219`	`+`
`215`	`220`	`public function testToStringJoinsHeadersTogether()`
`216`	`221`	`{`
`217`	`222`	`$headers = new Headers();`