bug #37899 [Mailer] Support reply-to in SesApiAsyncAwsTransport (cvmiert)

fabpot · fabpot · commit af91bf849a0c · 2020-08-21T08:36:53.000+02:00
This PR was merged into the 5.1 branch. Discussion ---------- [Mailer] Support reply-to in SesApiAsyncAwsTransport | Q | A | ------------- | --- | Branch? | 5.1 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | N/A | License | MIT | Doc PR | N/A Enable sending `SendEmailRequest`s with reply-to addresses with `SesApiAsyncAwsTransport`. Commits ------- 163e961 [Mailer] Support reply-to in SesApiAsyncAwsTransport
diff --git a/src/Symfony/Component/Mailer/Bridge/Amazon/Tests/Transport/SesApiAsyncAwsTransportTest.php b/src/Symfony/Component/Mailer/Bridge/Amazon/Tests/Transport/SesApiAsyncAwsTransportTest.php
@@ -68,6 +68,7 @@ public function testSend()
             $this->assertSame('Fabien <fabpot@symfony.com>', $content['FromEmailAddress']);
             $this->assertSame('Hello There!', $content['Content']['Simple']['Body']['Text']['Data']);
             $this->assertSame('<b>Hello There!</b>', $content['Content']['Simple']['Body']['Html']['Data']);
+            $this->assertSame(['replyto-1@example.com', 'replyto-2@example.com'], $content['ReplyToAddresses']);
 
             $json = '{"MessageId": "foobar"}';
 
@@ -83,7 +84,8 @@ public function testSend()
             ->to(new Address('saif.gmati@symfony.com', 'Saif Eddin'))
             ->from(new Address('fabpot@symfony.com', 'Fabien'))
             ->text('Hello There!')
-            ->html('<b>Hello There!</b>');
+            ->html('<b>Hello There!</b>')
+            ->replyTo(new Address('replyto-1@example.com'), new Address('replyto-2@example.com'));
 
         $message = $transport->send($mail);
 
diff --git a/src/Symfony/Component/Mailer/Bridge/Amazon/Transport/SesApiAsyncAwsTransport.php b/src/Symfony/Component/Mailer/Bridge/Amazon/Transport/SesApiAsyncAwsTransport.php
@@ -86,6 +86,9 @@ protected function getRequest(SentMessage $message): SendEmailRequest
                 'Charset' => $email->getHtmlCharset(),
             ]);
         }
+        if ($emails = $email->getReplyTo()) {
+            $request['ReplyToAddresses'] = $this->stringifyAddresses($emails);
+        }
 
         return new SendEmailRequest($request);
     }

Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,9 @@ protected function getRequest(SentMessage $message): SendEmailRequest`
`86`	`86`	`'Charset' => $email->getHtmlCharset(),`
`87`	`87`	`]);`
`88`	`88`	`}`
	`89`	`+ if ($emails = $email->getReplyTo()) {`
	`90`	`+ $request['ReplyToAddresses'] = $this->stringifyAddresses($emails);`
	`91`	`+ }`
`89`	`92`
`90`	`93`	`return new SendEmailRequest($request);`
`91`	`94`	`}`