• 
      

    Log the time taken for e-mails to go out.

    Review Request #14367 — Created March 13, 2025 and submitted

    Information

    Djblets
    release-5.x

    Reviewers

    To help diagnose e-mail-related issues, EmailMessage.send() now logs
    the time taken for the e-mails to go out. This logs the recipient
    message and the subject, to help identify the message.

    This is all logged as debug logs, so in production this shouldn't create
    too much additional noise unless debug messages are needed. However, it
    will make it clear via warning and critical logs if there are delays
    with sending.

    Unit tests pass.

    Tested this with some e-mail sending code in Review Board, verifying the
    log messages.

    Summary ID
    Log the time taken for e-mails to go out.
    To help diagnose e-mail-related issues, `EmailMessage.send()` now logs the time taken for the e-mails to go out. This logs the recipient message and the subject, to help identify the message. This is all logged as debug logs, so in production this shouldn't create too much additional noise unless debug messages are needed. However, it will make it clear via warning and critical logs if there are delays with sending.
    1a2ee89bbd94db8bd5a4a61370bb1d2e1b9159f9
    maubin
    1. Ship It!
    2. 
        
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-5.x (a7d321c)