#14973: AdminEmailHandler doesn't include report.message
-------------------------------------+--------------------------------------
Reporter: jamstooks | Owner: nobody
Status: new | Milestone: 1.3
Component: Core framework | Version: 1.3-alpha
Resolution: | Keywords:
Stage: Accepted | Has_patch: 0
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 0 |
-------------------------------------+--------------------------------------
Changes (by russellm):
* needs_better_patch: => 0
* stage: Unreviewed => Accepted
* needs_tests: => 0
* needs_docs: => 0
Comment:
Historically, this hasn't been an issue because there was only one
circumstance under which an admin email would be sent -- an internal
server error.
With the introduction of logging, there are other circumstances, so yes,
it makes sense to include the record.message.
My inclination would be to put record.message in the subject of the
message, replacing the current use of record.path. (line 74, where the
subject is constructed).
This is finessing a 1.3 feature, so it's 1.3 targeted, but not release
blocking.
--
Ticket URL: <http://code.djangoproject.com/ticket/14973#comment:1>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to django-upda...@googlegroups.com.
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
