Add log timers around all outbound communication code.
Review Request #14371 — Created March 17, 2025 and updated — Latest diff uploaded
When networks act up, Review Board can hit issues with timeouts or
delayed processing of requests. It can sometimes be hard to diagnose
what's going on in these cases.In an effort to address this, we now have log timers around all code
invoking HTTP requests, sending e-mails, communicating with
repositories, publishing, and performing security checks.These are DEBUG logs in most cases, so administrators will only see new
log items if operations are taking too long (triggering the WARNING or
CRTICIAL levels).
All unit tests pass.
Tested sending e-mails, fetching branches and commits, viewing diffs,
and triggered a webhook.
reviewboard/hostingsvcs/base/http.py |
---|