#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 [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/deluge-dev/048.949471bf72355d681f77e1b083abcc02%40deluge-torrent.org.
