Code review comment for lp:~abentley/launchpad/no-original-to

Revision history for this message
Aaron Bentley (abentley) wrote :

= Summary =
Stop using X-Original-To header for dispatching incoming mail, because it's
wrong.

== Proposed fix ==
Remove the code that prefers X-Original-To as the header, falling back on the
code that uses To and Cc, which was already present. These code changes have
been manually applied to production by the Losas for the past two months.

== Pre-implementation notes ==
Preimplementation was with Thumper.

== Implementation details ==
Some driveby fixes-- email.Utils -> email.utils to fix lint complaints, and a
spelling correction in a doctest.

== Tests ==
bin/test -t incoming.txt

== Demo and Q/A ==
Hard to assess.

= Launchpad lint =

Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.

Linting changed files:
  lib/canonical/launchpad/mail/incoming.py
  lib/canonical/launchpad/doc/incomingmail.txt

« Back to merge proposal