Mail Notifications: Do not CC disabled user accounts
Needs ReviewPublic
Actions

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

Details

Reviewers

None

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T16033: "To:" and "CC:" Recipient Hint footers in mail notifications list disabled user accounts

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

Test Failures

Build Status

Buildable 1878
Build 1878: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	601 ms	PhabricatorCelerityTestCase::testCelerityMaps Assertion failed, expected 'true' (at PhabricatorCelerityTestCase.php:27): When this test fails, it means the Celerity resource map is out of date. Run `bin/celerity map` to rebuild it. ACTUAL VALUE
	772 ms	PhabricatorConduitTestCase::testConduitMethods 1 assertion passed.
	441 ms	PhabricatorInfrastructureTestCase::testApplicationsInstalled 1 assertion passed.
	467 ms	PhabricatorInfrastructureTestCase::testRejectMySQLNonUTF8Queries 1 assertion passed.
	493 ms	PhabricatorLibraryTestCase::testEverythingImplemented 1 assertion passed.
		View Full Test Results (1 Failed · 6 Passed)