bug #61335 [Mailer] [Resend] Add friendly name in the to field (antalaron)

fabpot · fabpot · commit 986d4e2e40ef · 2025-08-09T21:07:00.000+02:00
This PR was merged into the 7.3 branch. Discussion ---------- [Mailer] [Resend] Add friendly name in the `to` field | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | n/a | License | MIT This PR adds support for friendly email address formatting (e.g. "John Doe <john@example.com>") in the `to` field. Although Resend's API accepts this format natively, the current implementation of the bridge does not pass it through correctly — names are silently discarded and only raw email addresses are used. This fix ensures that display names are preserved and passed to the API as intended. Commits ------- f9c0a3d Add friendly name in the `to` field
diff --git a/src/Symfony/Component/Mailer/Bridge/Resend/Tests/Transport/ResendApiTransportTest.php b/src/Symfony/Component/Mailer/Bridge/Resend/Tests/Transport/ResendApiTransportTest.php
@@ -152,7 +152,7 @@ public function testSendForIdnDomains()
 
             $body = json_decode($options['body'], true);
             // to
-            $this->assertSame('kältetechnik@xn--kltetechnik-xyz-0kb.de', $body['to'][0]);
+            $this->assertSame('Kältetechnik Xyz <kältetechnik@xn--kltetechnik-xyz-0kb.de>', $body['to'][0]);
             // sender
             $this->assertStringContainsString('info@xn--kltetechnik-xyz-0kb.de', $body['from']);
             $this->assertStringContainsString('Kältetechnik Xyz', $body['from']);
diff --git a/src/Symfony/Component/Mailer/Bridge/Resend/Transport/ResendApiTransport.php b/src/Symfony/Component/Mailer/Bridge/Resend/Transport/ResendApiTransport.php
@@ -82,7 +82,7 @@ private function formatAddresses(array $addresses): array
     {
         $formattedAddresses = [];
         foreach ($addresses as $address) {
-            $formattedAddresses[] = $address->getEncodedAddress();
+            $formattedAddresses[] = $this->formatAddress($address);
         }
 
         if (\count($formattedAddresses) > 50) {
@@ -99,8 +99,8 @@ private function getPayload(Email $email, Envelope $envelope): array
             'to' => $this->formatAddresses($this->getRecipients($email, $envelope)),
             'subject' => $email->getSubject(),
         ];
-        if ($attachements = $this->prepareAttachments($email)) {
-            $payload['attachments'] = $attachements;
+        if ($attachments = $this->prepareAttachments($email)) {
+            $payload['attachments'] = $attachments;
         }
         if ($emails = $email->getReplyTo()) {
             $payload['reply_to'] = current($this->formatAddresses($emails));

Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,7 @@ private function formatAddresses(array $addresses): array`
`82`	`82`	`{`
`83`	`83`	`$formattedAddresses = [];`
`84`	`84`	`foreach ($addresses as $address) {`
`85`		`- $formattedAddresses[] = $address->getEncodedAddress();`
	`85`	`+ $formattedAddresses[] = $this->formatAddress($address);`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`if (\count($formattedAddresses) > 50) {`
`@@ -99,8 +99,8 @@ private function getPayload(Email $email, Envelope $envelope): array`
`99`	`99`	`'to' => $this->formatAddresses($this->getRecipients($email, $envelope)),`
`100`	`100`	`'subject' => $email->getSubject(),`
`101`	`101`	`];`
`102`		`- if ($attachements = $this->prepareAttachments($email)) {`
`103`		`- $payload['attachments'] = $attachements;`
	`102`	`+ if ($attachments = $this->prepareAttachments($email)) {`
	`103`	`+ $payload['attachments'] = $attachments;`
`104`	`104`	`}`
`105`	`105`	`if ($emails = $email->getReplyTo()) {`
`106`	`106`	`$payload['reply_to'] = current($this->formatAddresses($emails));`