#3382: Notifications-0.3: improve smtplib.SMTP.sendmail() error logging
----------------------------+---------------------------
Reporter: lalbornoz | Type: patch
Status: new | Priority: minor
Milestone: needs verified | Component: Plugin
Version: 2.0.3 | Keywords: notifications
----------------------------+---------------------------
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'>, {}
}}}
--
Ticket URL: <https://dev.deluge-torrent.org/ticket/3382>
Deluge <https://deluge-torrent.org/>
Deluge Project
--
You received this message because you are subscribed to the Google Groups
"Deluge Dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to deluge-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/deluge-dev/048.949471bf72355d681f77e1b083abcc02%40deluge-torrent.org.
