[Mime] Fix generate message id with named address

This commit is contained in:
Jibé Barth 2019-03-04 20:56:49 +01:00 committed by Fabien Potencier
parent 1ad6f6f319
commit 375ac9237f
2 changed files with 11 additions and 1 deletions

View File

@ -97,7 +97,7 @@ class Message extends RawMessage
}
if (!$headers->has('Message-ID')) {
$headers->addIdHeader('Message-ID', $this->generateMessageId($sender->toString()));
$headers->addIdHeader('Message-ID', $this->generateMessageId($sender->getAddress()));
}
// remove the Bcc field which should NOT be part of the sent message

View File

@ -17,6 +17,7 @@ use Symfony\Component\Mime\Header\Headers;
use Symfony\Component\Mime\Header\MailboxListHeader;
use Symfony\Component\Mime\Header\UnstructuredHeader;
use Symfony\Component\Mime\Message;
use Symfony\Component\Mime\NamedAddress;
use Symfony\Component\Mime\Part\TextPart;
class MessageTest extends TestCase
@ -90,6 +91,15 @@ class MessageTest extends TestCase
(new Message())->getPreparedHeaders();
}
public function testGetPreparedHeadersWithNamedFrom()
{
$message = new Message();
$message->getHeaders()->addMailboxListHeader('From', [new NamedAddress('fabien@symfony.com', 'Fabien')]);
$h = $message->getPreparedHeaders();
$this->assertEquals(new MailboxListHeader('From', [new NamedAddress('fabien@symfony.com', 'Fabien')]), $h->get('From'));
$this->assertTrue($h->has('Message-Id'));
}
public function testGetPreparedHeadersHasSenderWhenNeeded()
{
$message = new Message();