HomePhorge

Begin improving the soundness of received mail

Description

Begin improving the soundness of received mail

Summary:
We/I broke a couple of things here recently (see D5911) and are doing some work here in general (see D5912, etc.).

Generally, this code is pretty oldschool and not especially well architected for modern application-oriented Phabricator. It hardcodes a lot of stuff which should be applications' responsibilites.

Take the first steps toward making it more solid to reduce the risk here. In particular:

  • Factor out the "self mail" and "duplicate mail" checks and add unit tests.
  • Make Message-ID hash handling automatic.

Test Plan: Ran unit tests.

Reviewers: btrahan, chad

Reviewed By: btrahan

CC: aran

Differential Revision: https://secure.phabricator.com/D5915

Details

Provenance
epriestleyAuthored on May 13 2013, 4:32 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP99f648e4ebf2: Don't show empty attributes.
Branches
Loading...
Tags
Loading...

Event Timeline