Page MenuHomePhorge

Mail Notifications: Do not CC disabled user accounts
Needs ReviewPublic

Authored by aklapper on Mon, Apr 21, 19:51.

Details

Summary

Do not list disabled user accounts in mail footer (when metamta.recipients.show-hints is enabled) and do not add them to the X-Phabricator-Cc header, as they are not supposed to receive mail anyway per https://we.phorge.it/source/phorge/browse/master/src/applications/metamta/query/PhabricatorMetaMTAActor.php;07723b46274c6dbf062af070e5c19101537e0d91$130-131.

Closes T16033

Test Plan

Likely trigger notifications on a task or changeset with disabled user accounts being subscribed or members/watchers of a project. Not tested though as I do not have a local mail setup.

Diff Detail

Repository
rP Phorge
Branch
T16033disabledRecipients (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1880
Build 1880: arc lint + arc unit