From 1a40d7c75a7a1b8dfc5e8c7a8054afaa208404f6 Mon Sep 17 00:00:00 2001 From: Oskar Stark Date: Fri, 11 Dec 2020 14:18:32 +0100 Subject: [PATCH] [Mailer] [Sendgrid] Use $scheme variable to stay consistent --- .../Bridge/Sendgrid/Transport/SendgridTransportFactory.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/Symfony/Component/Mailer/Bridge/Sendgrid/Transport/SendgridTransportFactory.php b/src/Symfony/Component/Mailer/Bridge/Sendgrid/Transport/SendgridTransportFactory.php index a4734c7213..22f56cbc85 100644 --- a/src/Symfony/Component/Mailer/Bridge/Sendgrid/Transport/SendgridTransportFactory.php +++ b/src/Symfony/Component/Mailer/Bridge/Sendgrid/Transport/SendgridTransportFactory.php @@ -23,16 +23,17 @@ final class SendgridTransportFactory extends AbstractTransportFactory { public function create(Dsn $dsn): TransportInterface { + $scheme = $dsn->getScheme(); $key = $this->getUser($dsn); - if ('sendgrid+api' === $dsn->getScheme()) { + if ('sendgrid+api' === $scheme) { $host = 'default' === $dsn->getHost() ? null : $dsn->getHost(); $port = $dsn->getPort(); return (new SendgridApiTransport($key, $this->client, $this->dispatcher, $this->logger))->setHost($host)->setPort($port); } - if ('sendgrid+smtp' === $dsn->getScheme() || 'sendgrid+smtps' === $dsn->getScheme() || 'sendgrid' === $dsn->getScheme()) { + if ('sendgrid+smtp' === $scheme || 'sendgrid+smtps' === $scheme || 'sendgrid' === $scheme) { return new SendgridSmtpTransport($key, $this->dispatcher, $this->logger); }