• 
      

    Add better control of From headers for e-mails.

    Review Request #10419 — Created Feb. 18, 2019 and submitted

    Information

    Review Board
    release-3.0.x

    Reviewers

    This builds upon the new work in Djblets and the contribution from Brian
    LeBlanc to add control over whether a Review Board server always uses
    a user's own e-mail address as the From address, never does, or chooses
    based on DMARC records.

    In previous releases, we had a Django setting for switching between
    smart spoofing and always spoofing, but never had any UI support for
    this, nor did we have any ability to force the default server address to
    be used. Brian's change gave users the ability to use the default server
    address through a new checkbox in the E-mail Settings page, but removed
    the ability to force spoofing (useful and faster in controlled
    environments).

    This change switches the checkbox to a drop-down list, giving the
    options to automatically choose whether to spoof, always spoof, or never
    spoof. It utilizes the new from_spoofing= argument in Djblets's
    EmailMessage class, replacing the old enable_smart_spoofing argument
    and settings.EMAIL_ENABLE_SMART_SPOOFING setting.

    It also renames the "Sender e-mail address" setting to
    "Default From address", which is a little more clear, especially alongside
    the new option.

    Unit tests passed.

    Manually tested with all the options, checking the headers of the
    resulting e-mails.

    Summary ID
    Add better control of From headers for e-mails.
    This builds upon the new work in Djblets and the contribution from Brian LeBlanc to add control over whether a Review Board server always uses a user's own e-mail address as the From address, never does, or chooses based on DMARC records. In previous releases, we had a Django setting for switching between smart spoofing and always spoofing, but never had any UI support for this, nor did we have any ability to force the default server address to be used. Brian's change gave users the ability to use the default server address through a new checkbox in the E-mail Settings page, but removed the ability to force spoofing (useful and faster in controlled environments). This change switches the checkbox to a drop-down list, giving the options to automatically choose whether to spoof, always spoof, or never spoof. It utilizes the new `from_spoofing=` argument in Djblets's `EmailMessage` class, replacing the old `enable_smart_spoofing` argument and `settings.EMAIL_ENABLE_SMART_SPOOFING` setting.
    96fbe15ef78c9f053da2af3b63a1a90960026089

    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-3.0.x (9579f29)