Page MenuHomePhabricator

Simplify some more older mail error handling code
ClosedPublic

Authored by epriestley on Apr 4 2014, 4:33 PM.
Tags
None
Referenced Files
F14059868: D8701.diff
Sun, Nov 17, 10:08 PM
F14046883: D8701.diff
Thu, Nov 14, 12:33 AM
F14034238: D8701.diff
Sat, Nov 9, 11:08 PM
F14018739: D8701.diff
Tue, Nov 5, 6:53 PM
F14003167: D8701.id20636.diff
Sat, Oct 26, 3:02 AM
F13998981: D8701.id20636.diff
Thu, Oct 24, 12:09 PM
F13998898: D8701.diff
Thu, Oct 24, 11:43 AM
F13998391: D8701.diff
Thu, Oct 24, 9:18 AM
Subscribers

Details

Summary

Ref T4371. We can reuse more code for this "your stuff is empty" error, now, and benefit from global rate limiting and being able to reply to arbitrary addresses.

Test Plan

Sent valid, empty, and empty-ignored email via mail_handler.php, got appropriate actions/errors/states.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Simplify some more older mail error handling code.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 4 2014, 5:50 PM
epriestley updated this revision to Diff 20643.

Closed by commit rPc8cf7bb5061c (authored by @epriestley).