From f4be5ffb5d3da675000e84d4b62ec38491bef520 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=93=D0=B5=D1=80=D1=85=D0=B0=D1=80=D0=B4=20PICCORO=20Len?= =?UTF-8?q?z=20McKAY?= Date: Thu, 7 Aug 2025 11:42:17 -0400 Subject: [PATCH] Fix workaround for #17491 log error on failed response for mail sending * Part of bunch of fixes, this fix #17491 where admins at test install cannot see the log of errors for UI test mail button, we can just see that this is the correct form cos other parts of the code manage the exception inside the catch using log interface class --- app/Http/Controllers/Api/SettingsController.php | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/app/Http/Controllers/Api/SettingsController.php b/app/Http/Controllers/Api/SettingsController.php index 7eb28a4815..286d5a8df6 100644 --- a/app/Http/Controllers/Api/SettingsController.php +++ b/app/Http/Controllers/Api/SettingsController.php @@ -150,8 +150,11 @@ class SettingsController extends Controller if (!config('app.lock_passwords')) { try { Notification::send(Setting::first(), new MailTest()); + Log::debug('Mail sending to '.config('mail.reply_to.address')); return response()->json(['message' => 'Mail sent to '.config('mail.reply_to.address')], 200); } catch (\Exception $e) { + Log::debug($e); + Log::error('Mail sent to '.config('mail.reply_to.address') . $e->getMessage()); return response()->json(['message' => $e->getMessage()], 500); } } @@ -315,4 +318,4 @@ class SettingsController extends Controller } -} \ No newline at end of file +}