feedback_email management command needs better exception handling

[ietf-svn-bot]

owner:jennifer@painless-security.com resolution_fixed type_defect | by rjsparks@nostrum.com

---

We have at least one instance of an email crashing the feedback_email command in a way that causes them to get a bounce from the mail system that looks like:

This is the mail system at host ietfa.amsl.com. I'm sorry to have to 
inform you that your message could not be delivered to one or more 
recipients. It's attached below. For further assistance, please send 
mail to postmaster. If you do so, please include this problem report. 
You can delete your own text from the attached returned message. The 
mail system <nomcom-recipients-21@ietfa.amsl.com>: Command died with 
status 1: "/a/www/ietf-datatracker/web/ietf/virtualenv-manage.py 
feedback_email --nomcom-year 2021". Command output: CommandError: 
'utf-8' codec can't decode byte 0xb7 in position 114: invalid start byte

This should not fail this way - if we can't recover enough to let the automated ingestion succeed it would be better to wrap the exception into something that gets sent (including the original message) to the nomcom chair.

---

Issue migrated from trac:3356 at 2022-03-04 09:08:46 +0000

[ietf-svn-bot]

@rjsparks@nostrum.com changed status from new to accepted

[ietf-svn-bot]

@jennifer@painless-security.com commented

---

On #3357, email to settings.ADMINS was requested. Should the admins be contacted for this one, either instead of or alongside the nomcom chair?

[ietf-svn-bot]

@rjsparks@nostrum.com commented

---

No - only the nomcom chair should get this one. These messages have confidential content that the admins should not see.

[ietf-svn-bot]

@jennifer@painless-security.com changed status from accepted to assigned

[ietf-svn-bot]

@jennifer@painless-security.com set owner to jennifer@painless-security.com

[ietf-svn-bot]

@jennifer@painless-security.com changed status from assigned to closed

[ietf-svn-bot]

@jennifer@painless-security.com set resolution to fixed

[ietf-svn-bot]

@jennifer@painless-security.com commented

---

Fixed in 968b775:

Create management command base class that sends emails on exceptions. Fixes #3356 and #3357. Commit ready for merge.

[ietf-svn-bot]

@rjsparks@nostrum.com commented

---

Fixed in d9dc4f5:

Merged in 968b775 from jennifer@painless-security.com:
Create management command base class that sends emails on exceptions. Fixes #3356 and #3357.