fix: prevent cascading crash in error email handler when SES not configured
This commit is contained in:
parent
36ed7dd9cf
commit
67eac2f1c9
1 changed files with 11 additions and 5 deletions
|
|
@ -49,12 +49,18 @@ defmodule Towerops.HoneybadgerNoticeFilter do
|
|||
|> subject(subject)
|
||||
|> text_body(body)
|
||||
|
||||
case Mailer.deliver(email) do
|
||||
{:ok, _} ->
|
||||
:ok
|
||||
try do
|
||||
case Mailer.deliver(email) do
|
||||
{:ok, _} ->
|
||||
:ok
|
||||
|
||||
{:error, reason} ->
|
||||
Logger.error("Failed to send Honeybadger error email: #{inspect(reason)}")
|
||||
{:error, reason} ->
|
||||
Logger.warning("Failed to send error email: #{inspect(reason)}")
|
||||
:error
|
||||
end
|
||||
rescue
|
||||
e ->
|
||||
Logger.warning("Error email delivery crashed: #{Exception.message(e)}")
|
||||
:error
|
||||
end
|
||||
end
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue