revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-01-02 02:40:58 +01:00
Code Issues Releases Wiki Activity

Add an explicit "this mail came from Phabricator" header

Browse source 
Summary:
See T926. If you want to write a mail rule that, e.g., captures Differential
mail but ignores people replying to it, it's kind of tricky right now. You can
use the 'X-Mail-Transport-Agent' header but that's not obvious and it's not
necessarily stable.

Add a nice, obvious "X-Phabricator-Sent-This-Message" header.

Test Plan: Sent myself some mail, verified the header appeared.

Reviewers: vrana, btrahan, fugalh, jungejason

Reviewed By: jungejason

CC: aran, epriestley

Maniphest Tasks: T926

Differential Revision: https://secure.phabricator.com/D1732
This commit is contained in:
epriestley 2012-02-29 15:30:56 -08:00
parent f6d4cc4896
commit 94daf63ca5
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/metamta/storage/mail/PhabricatorMetaMTAMail.php
View file

@ -440,9 +440,9 @@ class PhabricatorMetaMTAMail extends PhabricatorMetaMTADAO {
} }
} }
$mailer->addHeader('X-Phabricator-Sent-This-Message', 'Yes');
$mailer->addHeader('X-Mail-Transport-Agent', 'MetaMTA'); $mailer->addHeader('X-Mail-Transport-Agent', 'MetaMTA');
// If the message has mailtags, filter out any recipients who don't want // If the message has mailtags, filter out any recipients who don't want
// to receive this type of mail. // to receive this type of mail.
$mailtags = $this->getParam('mailtags'); $mailtags = $this->getParam('mailtags');