Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#14973 closed (fixed)

AdminEmailHandler doesn't include report.message

Reported by: jamstooks Owned by: nobody
Component: Core (Other) Version: 1.3-alpha
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

Shouldn't the message from the report be included in the email?

# django/utils/log.py line 89
message = "%s\n\n%s" % (stack_trace, request_repr)

I wasn't sure what the intension here was for excluding the message, so I didn't build a patch for that yet, but it could be something as simple as:

message = "%s\n\n%s\n\n%s" % (record.message, stack_trace, request_repr)

Change History (4)

comment:1 Changed 6 years ago by Russell Keith-Magee

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted

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.

comment:2 Changed 6 years ago by Russell Keith-Magee

Resolution: fixed
Status: newclosed

(In [15132]) Fixed #14973 -- Tweaked the email log handler so that the log message is actually reported.

comment:3 Changed 5 years ago by Jacob

milestone: 1.3

Milestone 1.3 deleted

comment:11 Changed 5 years ago by Jannis Leidel

In [17512]:

Fixed AdminEmailHandler to format the subject message correctly when arguments are passed.

This bug was hidden before r17480 as it basically didn't take the arguments into account. Refs #14973.

Note: See TracTickets for help on using tickets.
Back to Top