From 212d30cea4f56617ac4c0fa41585da1ebb955ddd Mon Sep 17 00:00:00 2001
From: shafeq <shafeq.hasan@gmail.com>
Date: Fri, 27 Feb 2026 19:11:08 +0800
Subject: [PATCH] fix: respect SMTP encryption setting for email notifications

The EsmtpTransport constructor expects bool|null for the tls parameter:
- null: auto-detect (try STARTTLS if supported)
- true: force implicit TLS (port 465)
- false: no encryption

Previously 'none' mapped to null which still attempted STARTTLS,
and 'tls' mapped to the string 'tls' instead of boolean true.

Fixes #6442
---
 app/Notifications/Channels/EmailChannel.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/Notifications/Channels/EmailChannel.php b/app/Notifications/Channels/EmailChannel.php
index abd115550..b986def07 100644
--- a/app/Notifications/Channels/EmailChannel.php
+++ b/app/Notifications/Channels/EmailChannel.php
@@ -80,8 +80,8 @@ class EmailChannel
             } elseif ($isSmtpEnabled) {
                 $encryption = match (strtolower($settings->smtp_encryption)) {
                     'starttls' => null,
-                    'tls' => 'tls',
-                    'none' => null,
+                    'tls' => true,
+                    'none' => false,
                     default => null,
                 };