Review Request #12705 — Created Oct. 28, 2022 and submitted
We have a special logging formatter that is used to show info when a
HttpRequestis part of the log record. Once upon a time, the
way of getting extra information into a log record was just to pass it
kwargsto the log methods, but the modern, documented
way of doing it is to pass a single
extradict. This change fixes our
code up to use that. We also had several logging instances where we had
the request available but were not passing it to the log.
While I was auditing our log messages, I fixed up some other trivial
issues like formatting with args and indentation.
Ran unit tests.