Opened 5 years ago
#3382 new patch
Notifications-0.3: improve smtplib.SMTP.sendmail() error logging
Reported by: | Lucio Andrés Illanes Albornoz | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | needs verified |
Component: | Plugin | Version: | 2.0.3 |
Keywords: | notifications | Cc: |
Description
The Notifications plugin as of version 0.3 only includes the value of an smtplib.SMTPException implicitly converted to a string in the log file on error:
Apr 11 15:06:08 [REDACTED] deluged[31554]: 15:06:08 [ERROR ][deluge_notifications.core :160 ] There was an error sending the notification email: {}
The patch enclosed includes both the exception type inasmuch as value in the logged error to aid debugging, e.g. yielding:
Apr 11 15:19:41 [REDACTED] deluged[539]: 15:19:41 [ERROR ][deluge_notifications.core :162 ] There was an error sending the notification email: <class 'smtplib.SMTPRecipientsRefused'>, {}
Attachments (1)
Change History (1)
by , 5 years ago
Attachment: | deluge-2.0.3+smtp_errors_Notifications_plugin.patch added |
---|
Note:
See TracTickets
for help on using tickets.