SendEmail will catch any errors thrown in Send, so Notifications don't blow up. It will also log additional details.

13 years ago · e8122685b2
parent b626dc2435
commit e8122685b2
1 changed files with 8 additions and 1 deletions
--- a/NzbDrone.Core/Providers/SmtpProvider.cs
+++ b/NzbDrone.Core/Providers/SmtpProvider.cs
@ -54,7 +54,14 @@ namespace NzbDrone.Core.Providers
                credentials = new NetworkCredential(username, password);

            //Send the email
-            Send(email, _configProvider.SmtpServer, _configProvider.SmtpPort, _configProvider.SmtpUseSsl, credentials);
+            try
+            {
+                Send(email, _configProvider.SmtpServer, _configProvider.SmtpPort, _configProvider.SmtpUseSsl, credentials);
+            }
+            catch(Exception ex)
+            {
+                Logger.Error("Error sending email. Subject: {0}", email.Subject);
+            }
        }

        public virtual bool SendTestEmail(string server, int port, bool ssl, string username, string password, string fromAddress, string toAddresses)