From bab77d38231497888f27b32f1d6f8addf4000586 Mon Sep 17 00:00:00 2001 From: Dave Mc Nicoll Date: Tue, 14 Jul 2026 20:36:08 +0000 Subject: [PATCH] - Fixed a bug where an empty hostname provided would cause a crash --- src/Email/SymfonyMailer.php | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/Email/SymfonyMailer.php b/src/Email/SymfonyMailer.php index ff46b3e..3cbb340 100644 --- a/src/Email/SymfonyMailer.php +++ b/src/Email/SymfonyMailer.php @@ -14,11 +14,13 @@ class SymfonyMailer implements MailerInterface public function __construct(EmailConfiguration $configuration) { $this->emailConfiguration = $configuration; - if ($this->emailConfiguration->smtpUsername) { - $this->mailer = new Mailer(Transport::fromDsn(sprintf("smtp://%s:%s@%s:%d?verify_peer=%s", $this->emailConfiguration->smtpUsername, $this->emailConfiguration->smtpPassword, $this->emailConfiguration->smtpHost, $this->emailConfiguration->smtpPort, $this->emailConfiguration->smtpVerifyPeer ? 'true' : 'false'))); - } - else { - $this->mailer = new Mailer(Transport::fromDsn(sprintf("smtp://%s:%d?verify_peer=%s", $this->emailConfiguration->smtpHost, $this->emailConfiguration->smtpPort, $this->emailConfiguration->smtpVerifyPeer ? 'true' : 'false'))); + if ($this->emailConfiguration->smtpHost) { + if ($this->emailConfiguration->smtpUsername) { + $this->mailer = new Mailer(Transport::fromDsn(sprintf("smtp://%s:%s@%s:%d?verify_peer=%s", $this->emailConfiguration->smtpUsername, $this->emailConfiguration->smtpPassword, $this->emailConfiguration->smtpHost, $this->emailConfiguration->smtpPort, $this->emailConfiguration->smtpVerifyPeer ? 'true' : 'false'))); + } + else { + $this->mailer = new Mailer(Transport::fromDsn(sprintf("smtp://%s:%d?verify_peer=%s", $this->emailConfiguration->smtpHost, $this->emailConfiguration->smtpPort, $this->emailConfiguration->smtpVerifyPeer ? 'true' : 'false'))); + } } }