Fix initial default for mail items, so that they're not assumed to be ham

[rjl]

When a mail item is first received by amavisd, it gets assigned a default type of "H" (ham), before being handed off to be classified further. Its type is changed only if it is later determined to be something other than ham.

While this system works well most of the time, it does present one subtle problem from time to time: a user checking his ham cache via the Maia interface at just the wrong moment may see an item listed as ham, just because at that moment the item has not yet made it to amavisd's classification routines. Seeing something that is "obviously" spam in his ham cache, he's likely to report it as a false negative, when in fact if he had simply refreshed his ham cache a second later, that item would have vanished (and correctly shown up on the spam quarantine list).

The easiest solution is to assign a neutral default type (e.g. "X"), so that such items will not be mistaken for ham prematurely. At the end of amavisd's classification system, any items that are still of type "X" should then be promoted to type "H".

[dmorton]

Or... When items are submitted, check its type first and do the stats appropriately. Though this route would add more database calls.

[dmorton]

looks like this is easier than I thought. I think all we have to do is

change the default. It appears is always updated in all code paths. The only corner cases I can think of are @. messages ... but I think they are covered to.

line 9637 of revision 1126 has the final conversion.

I made the change locally and I'm watching the database to see if any slip past.

[dmorton]

slipped into [1132] which was for another ticket, but it was a one liner.

Works great on my system.