From 7b3f7ea8f7beb212223b912d08844d1dbbe35a4f Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 31 Oct 2012 09:57:40 -0700 Subject: [PATCH] Don't send the address verification email "From" the user in question Summary: Sending these as the user doesn't make a ton of sense, and LLVM reports some issues with these emails getting caught in spam filters. Users expect these emails, so just send them from "noreply@example.com" or whatever is configured. Test Plan: Sent myself a verification email, verified it came from a noreply@ address. Reviewers: btrahan, vrana Reviewed By: btrahan CC: klimek, aran Maniphest Tasks: T1994 Differential Revision: https://secure.phabricator.com/D3843 --- src/applications/people/storage/PhabricatorUserEmail.php | 1 - 1 file changed, 1 deletion(-) diff --git a/src/applications/people/storage/PhabricatorUserEmail.php b/src/applications/people/storage/PhabricatorUserEmail.php index 17b6423b2a..3daa8faf9a 100644 --- a/src/applications/people/storage/PhabricatorUserEmail.php +++ b/src/applications/people/storage/PhabricatorUserEmail.php @@ -137,7 +137,6 @@ EOBODY; ->addRawTos(array($address)) ->setSubject('[Phabricator] Email Verification') ->setBody($body) - ->setFrom($user->getPHID()) ->setRelatedPHID($user->getPHID()) ->saveAndSend();