Page MenuHomePhabricator

Add repository to audit emails

Authored by joshuaspence on May 25 2015, 11:40 AM.
Referenced Files
Tue, May 21, 3:12 AM
F13214889: D12997.id31338.diff
Fri, May 17, 1:36 PM
F13188241: D12997.diff
Sat, May 11, 5:07 AM
Unknown Object (File)
Tue, May 7, 8:21 AM
Unknown Object (File)
Mon, May 6, 7:49 PM
Unknown Object (File)
Fri, May 3, 12:53 PM
Unknown Object (File)
Fri, May 3, 8:05 AM
Unknown Object (File)
Sun, Apr 28, 12:36 AM



This is vaguely related to T5791. Add a "REPOSITORY" header to audit emails so that they can be filtered in Gmail.

Test Plan

Commented on an audit and used ./bin/mail show-outbound to inspect outbound email.

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Add repository to audit emails.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

I'll accept this as a default-disabled custom field after D12993, see T8295.

This revision now requires changes to proceed.May 25 2015, 11:41 AM
joshuaspence edited edge metadata.

Convert to custom field

This doesn't actually seem to work, presumably because D12993 was intended for Herald emails?

I think you need to use updateTransactionMailBody instead of buildApplicationTransactionMailBody.

The later one has a comment on it saying:
TODO: this is only used by Diffusion right now and everything is completely
faked since Diffusion doesn't use ApplicationTransactions yet. This should
get fleshed out as we have more use cases.

and i can't actually find where it is called...

I'll fix the buildOldJunk callsites.

joshuaspence edited edge metadata.

Rebase. Yep, it seems to work now!

epriestley edited edge metadata.
This revision is now accepted and ready to land.May 25 2015, 12:43 PM
This revision was automatically updated to reflect the committed changes.