• 
      

    Add log timers around all outbound communication code.

    Review Request #14371 — Created March 17, 2025 and submitted — Latest diff uploaded

    Information

    Review Board
    release-7.1.x

    Reviewers

    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.

    Commits

    Files