Julian Cowley wrote:
Hi Stephan,
I noticed that you changed i;ascii-numeric comparator recently (<http://hg.rename-it.nl/dovecot-1.2-sieve/rev/17fb38aee85f>) to correct a small bug. I have an interest in this because I use the comparator to check for SpamAssassin scores. Before the fix, this was all that was necessary to check for scores above a certain value:
header :value "ge" :comparator "i;ascii-numeric" "X-Spam-Score" "5"Since SpamAssassin includes negative scores and the i;ascii-numeric comparator doesn't recognize negative numbers, all negative scores are actually treated as positive infinity after the fix. Of course, this means mail which has a negative score then gets counted as spam.
Autsj!
Now an additional test is necessary, something like this:
allof(not header :matches "X-Spam-Score" "-*", header :value "ge" :comparator "i;ascii-numeric" "X-Spam-Score" "5")I thought I'd mention this because other people may get affected by this once 0.1.15 comes out.
I'll mention it in the release mail.
Here is a patch that makes further tests for the comparator between numeric and non-numeric values. Hope you can use it.
I a added it just now. If only more people would file their bugs and comments as ready-made test suite scripts :)
Regards,
Stephan.