fix: issue with smtp non-tls connections (flarum#4203)

luceos · web-flow · commit 65d8c165807a · 2025-02-27T18:34:31.000+01:00
diff --git a/framework/core/src/Mail/SmtpDriver.php b/framework/core/src/Mail/SmtpDriver.php
@@ -53,7 +53,7 @@ public function canSend(): bool
     public function buildTransport(SettingsRepositoryInterface $settings): TransportInterface
     {
         return $this->factory->create(new Dsn(
-            $settings->get('mail_encryption') === 'tls' ? 'smtps' : '',
+            $settings->get('mail_encryption') === 'tls' ? 'smtps' : 'smtp',
             $settings->get('mail_host'),
             $settings->get('mail_username'),
             $settings->get('mail_password'),

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ public function canSend(): bool`
`53`	`53`	`public function buildTransport(SettingsRepositoryInterface $settings): TransportInterface`
`54`	`54`	`{`
`55`	`55`	`return $this->factory->create(new Dsn(`
`56`		`- $settings->get('mail_encryption') === 'tls' ? 'smtps' : '',`
	`56`	`+ $settings->get('mail_encryption') === 'tls' ? 'smtps' : 'smtp',`
`57`	`57`	`$settings->get('mail_host'),`
`58`	`58`	`$settings->get('mail_username'),`
`59`	`59`	`$settings->get('mail_password'),`