AdminEmailHandler doesn't include report.message

[jamstooks]

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)

[Russell Keith-Magee]

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.

[Russell Keith-Magee]

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

[Jacob]

Milestone 1.3 deleted

[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.