Skip to content
This repository has been archived by the owner on Jun 10, 2020. It is now read-only.

Catch Net::SMTPServerBusy exceptions #32

Open
atruskie opened this issue Jan 24, 2016 · 0 comments
Open

Catch Net::SMTPServerBusy exceptions #32

atruskie opened this issue Jan 24, 2016 · 0 comments

Comments

@atruskie
Copy link
Member

https://github.com/QutBioacoustics/baw-workers/blob/master/lib/baw-workers/mail/mailer.rb

When many error emails are sent at once, the mailer can throw a Net::SMTPServerBusy.

This exception should be handled because:

  • Exception handling code (which triggers the mailer) should not throw exceptions itself
  • The SMTP exception is raised and logged (for example in the resque logs) rather than original exception which hides the problems
  • and ideally, we don't want emails dropped
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant