symfony
diff --git a/‎src/Symfony/Component/Mime/Message.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Mime/Message.php
+9-2Lines changed: 9 additions & 2 deletions b/‎src/Symfony/Component/Mime/Message.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Mime/Message.php
+9-2Lines changed: 9 additions & 2 deletions
diff --git a/‎src/Symfony/Component/Mime/Tests/MessageTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Mime/Tests/MessageTest.php
+67Lines changed: 67 additions & 0 deletions b/‎src/Symfony/Component/Mime/Tests/MessageTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Mime/Tests/MessageTest.php
+67Lines changed: 67 additions & 0 deletions
@@ -124,11 +124,18 @@ public function toIterable(): iterable
 
     public function ensureValidity(): void
     {
-        if (!$this->headers->has('To') && !$this->headers->has('Cc') && !$this->headers->has('Bcc')) {
+        $to = (null !== $header = $this->headers->get('To')) ? $header->getBody() : null;
+        $cc = (null !== $header = $this->headers->get('Cc')) ? $header->getBody() : null;
+        $bcc = (null !== $header = $this->headers->get('Bcc')) ? $header->getBody() : null;
+
+        if (!$to && !$cc && !$bcc) {
             throw new LogicException('An email must have a "To", "Cc", or "Bcc" header.');
         }
 
-        if (!$this->headers->has('From') && !$this->headers->has('Sender')) {
+        $from = (null !== $header = $this->headers->get('From')) ? $header->getBody() : null;
+        $sender = (null !== $header = $this->headers->get('Sender')) ? $header->getBody() : null;
+
+        if (!$from && !$sender) {
             throw new LogicException('An email must have a "From" or a "Sender" header.');
         }
 
 
@@ -276,4 +276,71 @@ public function testSymfonySerialize()
         $serialized = $serializer->serialize($e, 'json');
         $this->assertStringMatchesFormat($expectedJson, json_encode(json_decode($serialized), \JSON_PRETTY_PRINT | \JSON_UNESCAPED_SLASHES));
     }
+
+    /**
+     * @dataProvider ensureValidityProvider
+     */
+    public function testEnsureValidity(array $headers, ?string $exceptionClass, ?string $exceptionMessage)
+    {
+        if ($exceptionClass) {
+            $this->expectException($exceptionClass);
+            $this->expectExceptionMessage($exceptionMessage);
+        } else {
+            $this->expectNotToPerformAssertions();
+        }
+
+        $m = new Message();
+        foreach ($headers as $headerName => $headerValue) {
+            $m->getHeaders()->addMailboxListHeader($headerName, $headerValue);
+        }
+        $m->ensureValidity();
+    }
+
+    public function ensureValidityProvider()
+    {
+        return [
+            'Valid address fields' => [
+                [
+                    'To' => ['dummy@symfony.com'],
+                    'From' => ['test@symfony.com'],
+                ],
+                null,
+                null,
+            ],
+
+            'No destination address fields' => [
+                [
+                    'From' => ['test@symfony.com'],
+                ],
+                LogicException::class,
+                'An email must have a "To", "Cc", or "Bcc" header.',
+            ],
+
+            'Empty destination address fields' => [
+                [
+                    'To' => [],
+                    'From' => ['test@symfony.com'],
+                ],
+                LogicException::class,
+                'An email must have a "To", "Cc", or "Bcc" header.',
+            ],
+
+            'No originator fields' => [
+                [
+                    'To' => ['dummy@symfony.com'],
+                ],
+                LogicException::class,
+                'An email must have a "From" or a "Sender" header.',
+            ],
+
+            'Empty originator fields' => [
+                [
+                    'To' => ['dummy@symfony.com'],
+                    'From' => [],
+                ],
+                LogicException::class,
+                'An email must have a "From" or a "Sender" header.',
+            ],
+        ];
+    }
 }
Original file line number	Diff line number	Diff line change
`@@ -124,11 +124,18 @@ public function toIterable(): iterable`
`124`	`124`
`125`	`125`	`public function ensureValidity(): void`
`126`	`126`	`{`
`127`		`- if (!$this->headers->has('To') && !$this->headers->has('Cc') && !$this->headers->has('Bcc')) {`
	`127`	`+ $to = (null !== $header = $this->headers->get('To')) ? $header->getBody() : null;`
	`128`	`+ $cc = (null !== $header = $this->headers->get('Cc')) ? $header->getBody() : null;`
	`129`	`+ $bcc = (null !== $header = $this->headers->get('Bcc')) ? $header->getBody() : null;`
	`130`	`+`
	`131`	`+ if (!$to && !$cc && !$bcc) {`
`128`	`132`	`throw new LogicException('An email must have a "To", "Cc", or "Bcc" header.');`
`129`	`133`	`}`
`130`	`134`
`131`		`- if (!$this->headers->has('From') && !$this->headers->has('Sender')) {`
	`135`	`+ $from = (null !== $header = $this->headers->get('From')) ? $header->getBody() : null;`
	`136`	`+ $sender = (null !== $header = $this->headers->get('Sender')) ? $header->getBody() : null;`
	`137`	`+`
	`138`	`+ if (!$from && !$sender) {`
`132`	`139`	`throw new LogicException('An email must have a "From" or a "Sender" header.');`
`133`	`140`	`}`
`134`	`141`