mirror of
https://github.com/vee1e/mitmproxy.git
synced 2026-09-02 10:47:14 +00:00
d72b92bdff
#### Description
It's hard to debug errors raised in addon scripts during startup as only
a generic message is output on the console. Using logger.format() to
format errors that occurred during startup instead of only displaying
the LogRecord.msg improves the output if an exception is present by
showing the stack trace. An additional newline was added for better
readability.
Comparison with the load_error.py test script, before:
$ mitmproxy -s test/mitmproxy/data/addonscripts/load_error.py
Error logged during startup: Addon error:
After:
$ mitmproxy -s test/mitmproxy/data/addonscripts/load_error.py
Error logged during startup:
Addon error:
Traceback (most recent call last):
File "test/mitmproxy/data/addonscripts/load_error.py", line 2, in load
raise ValueError()
ValueError
Relates to issue #5935 and PR #6020
#### Checklist
- [ ] I have updated tests where applicable.
- I think the value of extending `test_errorcheck.py` for this behavior
is low and tightly couples the test to `logger.format()`
- [ ] I have added an entry to the CHANGELOG.
- #6020 didn't introduce a changelog entry, so I figured this won't need
one either
|
||
|---|---|---|
| .. | ||
| addons | ||
| contentviews | ||
| contrib | ||
| coretypes | ||
| io | ||
| net | ||
| platform | ||
| proxy | ||
| script | ||
| test | ||
| tools | ||
| utils | ||
| __init__.py | ||
| addonmanager.py | ||
| certs.py | ||
| command.py | ||
| command_lexer.py | ||
| connection.py | ||
| ctx.py | ||
| dns.py | ||
| eventsequence.py | ||
| exceptions.py | ||
| flow.py | ||
| flowfilter.py | ||
| hooks.py | ||
| http.py | ||
| log.py | ||
| master.py | ||
| options.py | ||
| optmanager.py | ||
| py.typed | ||
| tcp.py | ||
| tls.py | ||
| types.py | ||
| udp.py | ||
| version.py | ||
| websocket.py | ||