From e3b6057bf84fac3faa7fa816abd8199f85b71986 Mon Sep 17 00:00:00 2001 From: James Cole Date: Thu, 3 Dec 2015 11:30:43 +0100 Subject: [PATCH] Catch Swift exceptions and do a log only (instead of crashing) because the email message isn't actually critical. --- app/Http/Controllers/Auth/AuthController.php | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/app/Http/Controllers/Auth/AuthController.php b/app/Http/Controllers/Auth/AuthController.php index cd84b74d15..ca5ffcefc1 100644 --- a/app/Http/Controllers/Auth/AuthController.php +++ b/app/Http/Controllers/Auth/AuthController.php @@ -182,11 +182,15 @@ class AuthController extends Controller $email = Auth::user()->email; $address = route('index'); // send email. - Mail::send( - ['emails.registered-html', 'emails.registered'], ['address' => $address], function (Message $message) use ($email) { - $message->to($email, $email)->subject('Welcome to Firefly III! '); + try { + Mail::send( + ['emails.registered-html', 'emails.registered'], ['address' => $address], function (Message $message) use ($email) { + $message->to($email, $email)->subject('Welcome to Firefly III! '); + } + ); + } catch(\Swift_TransportException $e) { + Log::error($e->getMessage()); } - ); // set flash message Session::flash('success', 'You have registered successfully!');