minor #39463 [Mailer] [Amazon] Ensure "ext-simplexml" is loaded if SesApiTransport is used (OskarStark)

jderusse · jderusse · commit e96b0e7d4d9a · 2020-12-14T21:27:32.000+01:00
This PR was squashed before being merged into the 4.4 branch. Discussion ---------- [Mailer] [Amazon] Ensure "ext-simplexml" is loaded if SesApiTransport is used | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | --- | License | MIT | Doc PR | --- For reference: https://github.com/symfony/symfony/blob/150d8506e425212bbdc0e75b7d5fc4e6710f0852/src/Symfony/Component/Mailer/Bridge/Amazon/Transport/SesApiTransport.php#L66-L69 Commits ------- a40d137 [Mailer] [Amazon] Ensure "ext-simplexml" is loaded if SesApiTransport is used
diff --git a/src/Symfony/Component/Mailer/Bridge/Amazon/Transport/SesTransportFactory.php b/src/Symfony/Component/Mailer/Bridge/Amazon/Transport/SesTransportFactory.php
@@ -11,6 +11,7 @@
 
 namespace Symfony\Component\Mailer\Bridge\Amazon\Transport;
 
+use Symfony\Component\Mailer\Exception\LogicException;
 use Symfony\Component\Mailer\Exception\UnsupportedSchemeException;
 use Symfony\Component\Mailer\Transport\AbstractTransportFactory;
 use Symfony\Component\Mailer\Transport\Dsn;
@@ -31,6 +32,10 @@ public function create(Dsn $dsn): TransportInterface
         $port = $dsn->getPort();
 
         if ('ses+api' === $scheme) {
+            if (!\extension_loaded('simplexml')) {
+                throw new LogicException(sprintf('Cannot use "%s". Make sure you have "ext-simplexml" installed and enabled.', SesApiTransport::class));
+            }
+
             return (new SesApiTransport($user, $password, $region, $this->client, $this->dispatcher, $this->logger))->setHost($host)->setPort($port);
         }
 
