Victor, to be honest, I don't think this is an issue. https://codereview.appspot.com/66540043/diff/1/asyncio/base_events.py File asyncio/base_events.py (right): ...
10 years, 2 months ago
(2014-02-20 16:00:04 UTC)
#1
Victor, to be honest, I don't think this is an issue.
https://codereview.appspot.com/66540043/diff/1/asyncio/base_events.py
File asyncio/base_events.py (right):
https://codereview.appspot.com/66540043/diff/1/asyncio/base_events.py#newcode677
asyncio/base_events.py:677: logger.error('\n'.join(log_lines))
Absolutely no!
Exception object *must* go to the logger routines. Lots of code relies on it,
just one example: error monitoring services like airbrake or newrelic install
their own LoggingHandlers to intercept exceptions and report them. They format
tracebacks and exception chains themselves.
Issue 66540043: Log context after the exception in default_exception_handler()
Created 10 years, 2 months ago by haypo_gmail
Modified 10 years, 2 months ago
Reviewers: yselivanov
Base URL:
Comments: 1