send-bug-notifications.py will explode if the SMTP server refuses to send to a recipient

Bug #314420 reported by Graham Binns on 2009-01-06
12
Affects Status Importance Assigned to Milestone
Launchpad itself
Low
Colin Watson

Bug Description

If the SMTP server refuses to send to a recipient (see https://pastebin.canonical.com/12356/ for an example), send-bug-notifications.py will die without committing to the database any sent notifications. This causes the notifications to be resent ad infinitum and stops send-bug-notifications from running properly.

send-bug-notifications should be updated to handle exceptions from smtplib.sendmail (which is where refusals from the server get turned into exceptions) gracefully and still commit those notifications that have been sent successfully.

Related branches

Graham Binns (gmb) wrote :

Marking this Low since it doesn't occur often.

Changed in malone:
importance: Undecided → Low
status: New → Triaged
Graham Binns (gmb) wrote :

Marking this high and targeting it to 2.2.1 since elmo reports this has caused problems more than once.

Changed in malone:
importance: Low → High
milestone: none → 2.2.1
Changed in malone:
assignee: nobody → intellectronica
status: Triaged → In Progress
Changed in malone:
milestone: 2.2.1 → 2.2.2
Björn Tillenius (bjornt) wrote :

This has been marked in progress for over a month. Let's mark it Triaged until work on it begins again.

Changed in malone:
importance: High → Medium
milestone: 2.2.2 → none
status: In Progress → Triaged
Curtis Hovey (sinzui) on 2010-06-01
Changed in malone:
assignee: Tom Berger (intellectronica) → nobody
Graham Binns (gmb) on 2010-06-04
tags: added: story-better-bug-notification
Graham Binns (gmb) on 2010-08-11
tags: added: story-better-notification-sending
Gary Poster (gary) on 2011-01-21
tags: removed: story-better-bug-notification
Curtis Hovey (sinzui) on 2011-09-29
Changed in launchpad:
importance: Medium → Low
tags: added: email
removed: story-better-notification-sending
Colin Watson (cjwatson) on 2015-07-15
Changed in launchpad:
assignee: nobody → Colin Watson (cjwatson)
status: Triaged → In Progress
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Colin Watson (cjwatson) on 2015-08-06
tags: added: qa-untestable
removed: qa-needstesting
William Grant (wgrant) on 2015-08-10
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers