If Robert is happy with the ability to preserve the headers, but having the default behavior change, I'm satisfied and would commit the change. Perhaps I am being too compulsive here; I've been working on a few patches to the Subversion project, where the compatibility guarantees are very strict (some would argue too strict). I just don't like to change the interface of the existing plugin without support from one of the other core developers...
From this overly long thread, it seems like I'm the only person who finds the multiple spam headers useful. In a perfect world, spam is only checked on one server. But in the world I live in, spam checks may get run three times on a single message.
Thus, how about this:
- Apply the patch.
- Default to mode '1' - X-Old-...
- Someone set an alarm for 3 months, and we can revisit it then.Mode '1' seems to be the compromise solution.
Tangentially - I'm not sure the numbering scheme makes sense. Or if we should be using flags like that. They're hard to understand without documentation.
-R (somewhere off the coast of Greece)
