[Mailer] Add support for multiple mailers

fabpot · fabpot · commit e929ee86acc5 · 2019-09-01T21:00:51.000+02:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php
@@ -1555,8 +1555,17 @@ private function addMailerSection(ArrayNodeDefinition $rootNode)
                 ->arrayNode('mailer')
                     ->info('Mailer configuration')
                     ->{!class_exists(FullStack::class) && class_exists(Mailer::class) ? 'canBeDisabled' : 'canBeEnabled'}()
+                    ->validate()
+                        ->ifTrue(function ($v) { return isset($v['dsn']) && \count($v['transports']); })
+                        ->thenInvalid('"dsn" and "transports" cannot be used together.')
+                    ->end()
+                    ->fixXmlConfig('transport')
                     ->children()
-                        ->scalarNode('dsn')->defaultValue('smtp://null')->end()
+                        ->scalarNode('dsn')->defaultNull()->end()
+                        ->arrayNode('transports')
+                            ->useAttributeAsKey('name')
+                            ->prototype('scalar')->end()
+                        ->end()
                         ->arrayNode('envelope')
                             ->info('Mailer Envelope configuration')
                             ->children()
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
@@ -1977,7 +1977,12 @@ private function registerMailerConfiguration(array $config, ContainerBuilder $co
 
         $loader->load('mailer.xml');
         $loader->load('mailer_transports.xml');
-        $container->getDefinition('mailer.default_transport')->setArgument(0, $config['dsn']);
+        if (!\count($config['transports']) && null === $config['dsn']) {
+            $config['dsn'] = 'smtp://null';
+        }
+        $transports = $config['dsn'] ? ['main' => $config['dsn']] : $config['transports'];
+        $container->getDefinition('mailer.transports')->setArgument(0, $transports);
+        $container->getDefinition('mailer.default_transport')->setArgument(0, current($transports));
 
         $classToServices = [
             SesTransportFactory::class => 'mailer.transport_factory.amazon',
diff --git a/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer.xml b/src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer.xml
@@ -6,13 +6,18 @@
 
     <services>
         <service id="mailer.mailer" class="Symfony\Component\Mailer\Mailer">
-            <argument type="service" id="mailer.default_transport" />
+            <argument type="service" id="mailer.transports" />
             <argument type="service" id="messenger.default_bus" on-invalid="ignore" />
             <argument type="service" id="event_dispatcher" on-invalid="ignore" />
         </service>
         <service id="mailer" alias="mailer.mailer" />
         <service id="Symfony\Component\Mailer\MailerInterface" alias="mailer.mailer" />
 
+        <service id="mailer.transports" class="Symfony\Component\Mailer\Transports">
+            <factory service="mailer.transport_factory" method="fromStrings" />
+            <argument type="collection" /> <!-- transports -->
+        </service>
+
         <service id="mailer.transport_factory" class="Symfony\Component\Mailer\Transport">
             <argument type="tagged_iterator" tag="mailer.transport_factory" />
         </service>
@@ -24,7 +29,7 @@
         <service id="Symfony\Component\Mailer\Transport\TransportInterface" alias="mailer.default_transport" />
 
         <service id="mailer.messenger.message_handler" class="Symfony\Component\Mailer\Messenger\MessageHandler">
-            <argument type="service" id="mailer.default_transport" />
+            <argument type="service" id="mailer.transports" />
             <tag name="messenger.message_handler" />
         </service>
 
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/ConfigurationTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/ConfigurationTest.php
@@ -369,7 +369,8 @@ class_exists(SemaphoreStore::class) && SemaphoreStore::isSupported() ? 'semaphor
                 'scoped_clients' => [],
             ],
             'mailer' => [
-                'dsn' => 'smtp://null',
+                'dsn' => null,
+                'transports' => [],
                 'enabled' => !class_exists(FullStack::class) && class_exists(Mailer::class),
             ],
         ];
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Functional/MailerTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Functional/MailerTest.php
@@ -8,6 +8,7 @@
 use Symfony\Component\Mailer\Mailer;
 use Symfony\Component\Mailer\SentMessage;
 use Symfony\Component\Mailer\Transport\AbstractTransport;
+use Symfony\Component\Mailer\Transports;
 use Symfony\Component\Mime\Address;
 use Symfony\Component\Mime\Email;
 
@@ -43,7 +44,7 @@ public function __construct(EventDispatcherInterface $eventDispatcher, LoggerInt
                 $this->onDoSend = $onDoSend;
             }
 
-            public function getName(): string
+            public function __toString(): string
             {
                 return 'dummy://local';
             }
@@ -55,7 +56,7 @@ protected function doSend(SentMessage $message): void
             }
         };
 
-        $mailer = new Mailer($testTransport, null);
+        $mailer = new Mailer(new Transports(['main' => $testTransport]));
 
         $message = (new Email())
             ->subject('Test subject')
diff --git a/src/Symfony/Component/Mailer/Bridge/Amazon/Transport/SesApiTransport.php b/src/Symfony/Component/Mailer/Bridge/Amazon/Transport/SesApiTransport.php
@@ -43,7 +43,7 @@ public function __construct(string $accessKey, string $secretKey, string $region
         parent::__construct($client, $dispatcher, $logger);
     }
 
-    public function getName(): string
+    public function __toString(): string
     {
         return sprintf('api://%s@ses?region=%s', $this->accessKey, $this->region);
     }
diff --git a/src/Symfony/Component/Mailer/Bridge/Amazon/Transport/SesHttpTransport.php b/src/Symfony/Component/Mailer/Bridge/Amazon/Transport/SesHttpTransport.php
@@ -42,7 +42,7 @@ public function __construct(string $accessKey, string $secretKey, string $region
         parent::__construct($client, $dispatcher, $logger);
     }
 
-    public function getName(): string
+    public function __toString(): string
     {
         return sprintf('http://%s@ses?region=%s', $this->accessKey, $this->region);
     }
diff --git a/src/Symfony/Component/Mailer/Bridge/Mailchimp/Transport/MandrillApiTransport.php b/src/Symfony/Component/Mailer/Bridge/Mailchimp/Transport/MandrillApiTransport.php
@@ -36,7 +36,7 @@ public function __construct(string $key, HttpClientInterface $client = null, Eve
         parent::__construct($client, $dispatcher, $logger);
     }
 
-    public function getName(): string
+    public function __toString(): string
     {
         return sprintf('api://mandrill');
     }
diff --git a/src/Symfony/Component/Mailer/Bridge/Mailchimp/Transport/MandrillHttpTransport.php b/src/Symfony/Component/Mailer/Bridge/Mailchimp/Transport/MandrillHttpTransport.php
@@ -34,7 +34,7 @@ public function __construct(string $key, HttpClientInterface $client = null, Eve
         parent::__construct($client, $dispatcher, $logger);
     }
 
-    public function getName(): string
+    public function __toString(): string
     {
         return sprintf('http://mandrill');
     }
diff --git a/src/Symfony/Component/Mailer/Bridge/Mailgun/Transport/MailgunApiTransport.php b/src/Symfony/Component/Mailer/Bridge/Mailgun/Transport/MailgunApiTransport.php
@@ -41,7 +41,7 @@ public function __construct(string $key, string $domain, string $region = null,
         parent::__construct($client, $dispatcher, $logger);
     }
 
-    public function getName(): string
+    public function __toString(): string
     {
         return sprintf('api://%s@mailgun?region=%s', $this->domain, $this->region);
     }
diff --git a/src/Symfony/Component/Mailer/Bridge/Mailgun/Transport/MailgunHttpTransport.php b/src/Symfony/Component/Mailer/Bridge/Mailgun/Transport/MailgunHttpTransport.php
@@ -40,7 +40,7 @@ public function __construct(string $key, string $domain, string $region = null,
         parent::__construct($client, $dispatcher, $logger);
     }
 
-    public function getName(): string
+    public function __toString(): string
     {
         return sprintf('http://%s@mailgun?region=%s', $this->domain, $this->region);
     }
diff --git a/src/Symfony/Component/Mailer/Bridge/Postmark/Transport/PostmarkApiTransport.php b/src/Symfony/Component/Mailer/Bridge/Postmark/Transport/PostmarkApiTransport.php
@@ -36,7 +36,7 @@ public function __construct(string $key, HttpClientInterface $client = null, Eve
         parent::__construct($client, $dispatcher, $logger);
     }
 
-    public function getName(): string
+    public function __toString(): string
     {
         return sprintf('api://postmark');
     }
diff --git a/src/Symfony/Component/Mailer/Bridge/Sendgrid/Tests/Transport/SendgridTransportFactoryTest.php b/src/Symfony/Component/Mailer/Bridge/Sendgrid/Tests/Transport/SendgridTransportFactoryTest.php
@@ -76,4 +76,9 @@ public function unsupportedSchemeProvider(): iterable
             'The "foo" scheme is not supported for mailer "sendgrid". Supported schemes are: "api", "smtp", "smtps".',
         ];
     }
+
+    public function incompleteDsnProvider(): iterable
+    {
+        yield [new Dsn('api', 'sendgrid')];
+    }
 }
diff --git a/src/Symfony/Component/Mailer/Bridge/Sendgrid/Transport/SendgridApiTransport.php b/src/Symfony/Component/Mailer/Bridge/Sendgrid/Transport/SendgridApiTransport.php
@@ -37,7 +37,7 @@ public function __construct(string $key, HttpClientInterface $client = null, Eve
         parent::__construct($client, $dispatcher, $logger);
     }
 
-    public function getName(): string
+    public function __toString(): string
     {
         return sprintf('api://sendgrid');
     }
diff --git a/src/Symfony/Component/Mailer/CHANGELOG.md b/src/Symfony/Component/Mailer/CHANGELOG.md
@@ -4,14 +4,16 @@ CHANGELOG
 4.4.0
 -----
 
+ * added support for multiple transports on a `Mailer` instance
+ * [BC BREAK] changed the first argument of `Mailer` from `TransportInterface` to `Transports`
  * [BC BREAK] removed the `auth_mode` DSN option (it is now always determined automatically)
  * STARTTLS cannot be enabled anymore (it is used automatically if TLS is disabled and the server supports STARTTLS)
  * [BC BREAK] Removed the `encryption` DSN option (use `smtps` instead)
  * Added support for the `smtps` protocol (does the same as using `smtp` and port `465`)
  * Added PHPUnit constraints
  * Added `MessageDataCollector`
  * Added `MessageEvents` and `MessageLoggerListener` to allow collecting sent emails
- * [BC BREAK] `TransportInterface` has a new `getName()` method
+ * [BC BREAK] `TransportInterface` has a new `__toString()` method
  * [BC BREAK] Classes `AbstractApiTransport` and `AbstractHttpTransport` moved under `Transport` sub-namespace.
  * [BC BREAK] Transports depend on `Symfony\Contracts\EventDispatcher\EventDispatcherInterface`
    instead of `Symfony\Component\EventDispatcher\EventDispatcherInterface`.
diff --git a/src/Symfony/Component/Mailer/Mailer.php b/src/Symfony/Component/Mailer/Mailer.php
@@ -14,7 +14,6 @@
 use Symfony\Component\Mailer\Event\MessageEvent;
 use Symfony\Component\Mailer\Exception\TransportException;
 use Symfony\Component\Mailer\Messenger\SendEmailMessage;
-use Symfony\Component\Mailer\Transport\TransportInterface;
 use Symfony\Component\Messenger\MessageBusInterface;
 use Symfony\Component\Mime\RawMessage;
 use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
@@ -24,21 +23,32 @@
  */
 class Mailer implements MailerInterface
 {
-    private $transport;
+    private $transports;
     private $bus;
     private $dispatcher;
 
-    public function __construct(TransportInterface $transport, MessageBusInterface $bus = null, EventDispatcherInterface $dispatcher = null)
+    public function __construct(Transports $transports, MessageBusInterface $bus = null, EventDispatcherInterface $dispatcher = null)
     {
-        $this->transport = $transport;
+        $this->transports = $transports;
         $this->bus = $bus;
         $this->dispatcher = $dispatcher;
     }
 
-    public function send(RawMessage $message, SmtpEnvelope $envelope = null): void
+    public function getDefaultTransport(): string
     {
+        return $this->transports->getDefault();
+    }
+
+    public function send(RawMessage $message, SmtpEnvelope $envelope = null, string $transportName = null): void
+    {
+        if (null === $transportName) {
+            $transportName = $this->transports->getDefault();
+        }
+
+        $transport = $this->transports->get($transportName);
+
         if (null === $this->bus) {
-            $this->transport->send($message, $envelope);
+            $transport->send($message, $envelope);
 
             return;
         }
@@ -54,10 +64,10 @@ public function send(RawMessage $message, SmtpEnvelope $envelope = null): void
                     throw new TransportException('Cannot send message without a valid envelope.', 0, $e);
                 }
             }
-            $event = new MessageEvent($message, $envelope, $this->transport->getName(), true);
+            $event = new MessageEvent($message, $envelope, $transportName, true);
             $this->dispatcher->dispatch($event);
         }
 
-        $this->bus->dispatch(new SendEmailMessage($message, $envelope));
+        $this->bus->dispatch(new SendEmailMessage($message, $envelope, $transportName));
     }
 }
diff --git a/src/Symfony/Component/Mailer/MailerInterface.php b/src/Symfony/Component/Mailer/MailerInterface.php
@@ -26,5 +26,5 @@ interface MailerInterface
     /**
      * @throws TransportExceptionInterface
      */
-    public function send(RawMessage $message, SmtpEnvelope $envelope = null): void;
+    public function send(RawMessage $message, SmtpEnvelope $envelope = null, string $transport = null): void;
 }
diff --git a/src/Symfony/Component/Mailer/Messenger/MessageHandler.php b/src/Symfony/Component/Mailer/Messenger/MessageHandler.php
@@ -11,22 +11,22 @@
 
 namespace Symfony\Component\Mailer\Messenger;
 
-use Symfony\Component\Mailer\Transport\TransportInterface;
+use Symfony\Component\Mailer\Transports;
 
 /**
  * @author Fabien Potencier <fabien@symfony.com>
  */
 class MessageHandler
 {
-    private $transport;
+    private $transports;
 
-    public function __construct(TransportInterface $transport)
+    public function __construct(Transports $transports)
     {
-        $this->transport = $transport;
+        $this->transports = $transports;
     }
 
     public function __invoke(SendEmailMessage $message)
     {
-        $this->transport->send($message->getMessage(), $message->getEnvelope());
+        $this->transports->get($message->getTransport())->send($message->getMessage(), $message->getEnvelope());
     }
 }
diff --git a/src/Symfony/Component/Mailer/Messenger/SendEmailMessage.php b/src/Symfony/Component/Mailer/Messenger/SendEmailMessage.php
@@ -19,16 +19,23 @@
  */
 class SendEmailMessage
 {
+    private $transport;
     private $message;
     private $envelope;
 
     /**
      * @internal
      */
-    public function __construct(RawMessage $message, SmtpEnvelope $envelope = null)
+    public function __construct(RawMessage $message, SmtpEnvelope $envelope = null, string $transport = null)
     {
         $this->message = $message;
         $this->envelope = $envelope;
+        $this->transport = $transport;
+    }
+
+    public function getTransport(): ?string
+    {
+        return $this->transport;
     }
 
     public function getMessage(): RawMessage
diff --git a/src/Symfony/Component/Mailer/Test/TransportFactoryTestCase.php b/src/Symfony/Component/Mailer/Test/TransportFactoryTestCase.php
@@ -70,7 +70,7 @@ public function testCreate(Dsn $dsn, TransportInterface $transport): void
 
         $this->assertEquals($transport, $factory->create($dsn));
         if ('smtp' !== $dsn->getScheme() && 'smtps' !== $dsn->getScheme()) {
-            $this->assertStringMatchesFormat($dsn->getScheme().'://%S'.$dsn->getHost().'%S', $transport->getName());
+            $this->assertStringMatchesFormat($dsn->getScheme().'://%S'.$dsn->getHost().'%S', (string) $transport);
         }
     }
 
diff --git a/src/Symfony/Component/Mailer/Tests/Transport/FailoverTransportTest.php b/src/Symfony/Component/Mailer/Tests/Transport/FailoverTransportTest.php
@@ -29,14 +29,14 @@ public function testSendNoTransports()
         new FailoverTransport([]);
     }
 
-    public function testGetName()
+    public function testToString()
     {
         $t1 = $this->createMock(TransportInterface::class);
-        $t1->expects($this->once())->method('getName')->willReturn('t1://local');
+        $t1->expects($this->once())->method('__toString')->willReturn('t1://local');
         $t2 = $this->createMock(TransportInterface::class);
-        $t2->expects($this->once())->method('getName')->willReturn('t2://local');
+        $t2->expects($this->once())->method('__toString')->willReturn('t2://local');
         $t = new FailoverTransport([$t1, $t2]);
-        $this->assertEquals('t1://local || t2://local', $t->getName());
+        $this->assertEquals('t1://local || t2://local', (string) $t);
     }
 
     public function testSendFirstWork()
diff --git a/src/Symfony/Component/Mailer/Tests/Transport/NullTransportTest.php b/src/Symfony/Component/Mailer/Tests/Transport/NullTransportTest.php
@@ -16,9 +16,9 @@
 
 class NullTransportTest extends TestCase
 {
-    public function testName()
+    public function testToString()
     {
         $t = new NullTransport();
-        $this->assertEquals('smtp://null', $t->getName());
+        $this->assertEquals('smtp://null', (string) $t);
     }
 }
diff --git a/src/Symfony/Component/Mailer/Tests/Transport/RoundRobinTransportTest.php b/src/Symfony/Component/Mailer/Tests/Transport/RoundRobinTransportTest.php
@@ -28,14 +28,14 @@ public function testSendNoTransports()
         new RoundRobinTransport([]);
     }
 
-    public function testGetName()
+    public function testToString()
     {
         $t1 = $this->createMock(TransportInterface::class);
-        $t1->expects($this->once())->method('getName')->willReturn('t1://local');
+        $t1->expects($this->once())->method('__toString')->willReturn('t1://local');
         $t2 = $this->createMock(TransportInterface::class);
-        $t2->expects($this->once())->method('getName')->willReturn('t2://local');
+        $t2->expects($this->once())->method('__toString')->willReturn('t2://local');
         $t = new RoundRobinTransport([$t1, $t2]);
-        $this->assertEquals('t1://local && t2://local', $t->getName());
+        $this->assertEquals('t1://local && t2://local', (string) $t);
     }
 
     public function testSendAlternate()
diff --git a/src/Symfony/Component/Mailer/Tests/Transport/SendmailTransportTest.php b/src/Symfony/Component/Mailer/Tests/Transport/SendmailTransportTest.php
@@ -16,9 +16,9 @@
 
 class SendmailTransportTest extends TestCase
 {
-    public function testName()
+    public function testToString()
     {
         $t = new SendmailTransport();
-        $this->assertEquals('smtp://sendmail', $t->getName());
+        $this->assertEquals('smtp://sendmail', (string) $t);
     }
 }
diff --git a/src/Symfony/Component/Mailer/Tests/Transport/Smtp/EsmtpTransportTest.php b/src/Symfony/Component/Mailer/Tests/Transport/Smtp/EsmtpTransportTest.php
diff --git a/src/Symfony/Component/Mailer/Tests/Transport/Smtp/SmtpTransportTest.php b/src/Symfony/Component/Mailer/Tests/Transport/Smtp/SmtpTransportTest.php
diff --git a/src/Symfony/Component/Mailer/Tests/TransportTest.php b/src/Symfony/Component/Mailer/Tests/TransportTest.php
diff --git a/src/Symfony/Component/Mailer/Transport.php b/src/Symfony/Component/Mailer/Transport.php
diff --git a/src/Symfony/Component/Mailer/Transport/AbstractTransport.php b/src/Symfony/Component/Mailer/Transport/AbstractTransport.php
diff --git a/src/Symfony/Component/Mailer/Transport/NullTransport.php b/src/Symfony/Component/Mailer/Transport/NullTransport.php
diff --git a/src/Symfony/Component/Mailer/Transport/RoundRobinTransport.php b/src/Symfony/Component/Mailer/Transport/RoundRobinTransport.php
diff --git a/src/Symfony/Component/Mailer/Transport/SendmailTransport.php b/src/Symfony/Component/Mailer/Transport/SendmailTransport.php
diff --git a/src/Symfony/Component/Mailer/Transport/Smtp/SmtpTransport.php b/src/Symfony/Component/Mailer/Transport/Smtp/SmtpTransport.php
diff --git a/src/Symfony/Component/Mailer/Transport/TransportInterface.php b/src/Symfony/Component/Mailer/Transport/TransportInterface.php
diff --git a/src/Symfony/Component/Mailer/Transports.php b/src/Symfony/Component/Mailer/Transports.php

Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@`
`8`	`8`	`use Symfony\Component\Mailer\Mailer;`
`9`	`9`	`use Symfony\Component\Mailer\SentMessage;`
`10`	`10`	`use Symfony\Component\Mailer\Transport\AbstractTransport;`
	`11`	`+use Symfony\Component\Mailer\Transports;`
`11`	`12`	`use Symfony\Component\Mime\Address;`
`12`	`13`	`use Symfony\Component\Mime\Email;`
`13`	`14`
`@@ -43,7 +44,7 @@ public function __construct(EventDispatcherInterface $eventDispatcher, LoggerInt`
`43`	`44`	`$this->onDoSend = $onDoSend;`
`44`	`45`	`}`
`45`	`46`
`46`		`- public function getName(): string`
	`47`	`+ public function __toString(): string`
`47`	`48`	`{`
`48`	`49`	`return 'dummy://local';`
`49`	`50`	`}`
`@@ -55,7 +56,7 @@ protected function doSend(SentMessage $message): void`
`55`	`56`	`}`
`56`	`57`	`};`
`57`	`58`
`58`		`- $mailer = new Mailer($testTransport, null);`
	`59`	`+ $mailer = new Mailer(new Transports(['main' => $testTransport]));`
`59`	`60`
`60`	`61`	`$message = (new Email())`
`61`	`62`	`->subject('Test subject')`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ public function __construct(string $accessKey, string $secretKey, string $region`
`43`	`43`	`parent::__construct($client, $dispatcher, $logger);`
`44`	`44`	`}`
`45`	`45`
`46`		`- public function getName(): string`
	`46`	`+ public function __toString(): string`
`47`	`47`	`{`
`48`	`48`	`return sprintf('api://%s@ses?region=%s', $this->accessKey, $this->region);`
`49`	`49`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ public function __construct(string $accessKey, string $secretKey, string $region`
`42`	`42`	`parent::__construct($client, $dispatcher, $logger);`
`43`	`43`	`}`
`44`	`44`
`45`		`- public function getName(): string`
	`45`	`+ public function __toString(): string`
`46`	`46`	`{`
`47`	`47`	`return sprintf('http://%s@ses?region=%s', $this->accessKey, $this->region);`
`48`	`48`	`}`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ public function __construct(string $key, HttpClientInterface $client = null, Eve`
`36`	`36`	`parent::__construct($client, $dispatcher, $logger);`
`37`	`37`	`}`
`38`	`38`
`39`		`- public function getName(): string`
	`39`	`+ public function __toString(): string`
`40`	`40`	`{`
`41`	`41`	`return sprintf('api://mandrill');`
`42`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ public function __construct(string $key, HttpClientInterface $client = null, Eve`
`34`	`34`	`parent::__construct($client, $dispatcher, $logger);`
`35`	`35`	`}`
`36`	`36`
`37`		`- public function getName(): string`
	`37`	`+ public function __toString(): string`
`38`	`38`	`{`
`39`	`39`	`return sprintf('http://mandrill');`
`40`	`40`	`}`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public function __construct(string $key, string $domain, string $region = null,`
`41`	`41`	`parent::__construct($client, $dispatcher, $logger);`
`42`	`42`	`}`
`43`	`43`
`44`		`- public function getName(): string`
	`44`	`+ public function __toString(): string`
`45`	`45`	`{`
`46`	`46`	`return sprintf('api://%s@mailgun?region=%s', $this->domain, $this->region);`
`47`	`47`	`}`