Page MenuHomePhabricator

Don't send error/exception mail to unverified addresses
ClosedPublic

Authored by epriestley on Feb 7 2018, 1:18 PM.
Tags
None
Referenced Files
F13480336: D19018.diff
Fri, Jul 19, 8:10 AM
F13461146: D19018.diff
Mon, Jul 15, 12:04 PM
F13430781: D19018.diff
Wed, Jul 10, 6:32 AM
F13418321: D19018.id45614.diff
Mon, Jul 8, 12:55 AM
F13406909: D19018.diff
Fri, Jul 5, 10:23 PM
F13379247: D19018.id45587.diff
Sat, Jun 29, 5:48 AM
F13369363: D19018.id45587.diff
Thu, Jun 27, 11:11 AM
F13359015: D19018.diff
Tue, Jun 25, 8:52 AM
Subscribers
None

Details

Summary

Depends on D19017. Fixes T12491. Ref T13053. After SES threw us in the dungeon for sending mail to a spamtrap we changed outbound mail rules to stop sending to unverified addresses, except a small amount of registration mail which we can't avoid.

However, we'll still reply to random inbound messages with a helpful error, even if the sender is unverified.

Instead, only send exception mail back if we know who the sender is.

Test Plan

Processed inbound mail with scripts/mail/mail_handler.php. No more outbound mail for "bad address", etc. Still got outbound mail for "unknown command !quack".

Diff Detail

Repository
rP Phabricator
Branch
mailer16
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19381
Build 26212: Run Core Tests
Build 26211: arc lint + arc unit