Sieve: fileinto and addheader/tag

Question

I'm having a problem with a spam sieve script and fileinto. I'm not sure if there's something about sieve, fileinto or how it's implemented that I don't understand, but it seems like it doesn't work like I'd hope.
imta.cnf has this line:
ims-ms master_debug defragment subdirs 20 notices 1 7 14 21 28 backoff "pt5m" "pt10m" "pt30m" "pt1h" "pt2h" "pt4h" maxjobs 2 pool IMS_POOL fileinto $U+$S@$D destinationspamfilter1optin spam
spamassassin.opt:
host=127.0.0.1
port=783
mode=2
field=
debug=1
option.dat:
! SpamAssassin
spamfilter1_config_file=/opt/sun/comms/messaging64/config/spamassassin.opt
spamfilter1_library=/opt/sun/comms/messaging64/lib/libspamass.so
spamfilter1_optional=1
spamfilter1_string_action=data:, \
require ["spamtest","relational","comparator-i;ascii-numeric","fileinto","addheader"]; \
spamadjust "$U"; \
addheader "Spam-test: $U"; \
if spamtest :value "ge" :comparator "i;ascii-numeric" "6" { \
addtag "[SPAM_HI: $U]"; \
} elsif spamtest :value "ge" :comparator "i;ascii-numeric" "4" { \
addtag "[SPAM_LOW: $U]"; \
Basically, if the spamscore is < 6, add SPAM_HI to the subject. If between 6 and 4, add SPAM_LOW to the subject, otherwise, leave it alone. Always add a 'Spam-test:' header to the message. The above works fine.
Now what I really want to do is file the bad spam straight into the users mailbox. But I want to leave the tags and headers:
... same ...
if spamtest :value "ge" :comparator "i;ascii-numeric" "6" { \
addtag "[SPAM_HI: $U]"; \
fileinto "spam"; \
addtag "[SPAM_HI2: $U]"; \
} elsif spamtest :value "ge" :comparator "i;ascii-numeric" "4" { \
... same ...
And if the spamscore is 6 or higher, it's delivered to <user>+spam. If it's between 6 and 4, it gets a SPAM_LO tag, and if it's less than 6, it gets a header.
The problem is, the messages that are 6 or higher, and delivered to the spam subfolder have neither tags added or headers added. Obviously the if/then worked, or it wouldn't have been send to spam. The other parts of the if statement are working.
The problems seems to be that if you use fileinto, it's like it files the original message, before any addtag, or addheader commands. Anything you do to the message ends up being thrown away. In this case, I'd like to have the Spam-test: header on every message, so I can check what the score was on each message.
So, is fileinto supposed to work this way? I've tried reading the docs and RFCs, but I can't find anything that explicitly says it should work this way.
Thanks,
John
bash-3.2# ./imsimta version
Sun Java(tm) System Messaging Server 7.0-3.01 64bit (built Dec 9 2008)
libimta.so 7.0-3.01 64bit (built 09:24:13, Dec 9 2008)
Using /opt/sun/comms/messaging64/config/imta.cnf
SunOS mail 5.11 snv_104 i86pc i386 i86pc

Answer 1

shane_hjorth wrote:
jandrusiak wrote:
I'm having a problem with a spam sieve script and fileinto. I'm not sure if there's something about sieve, fileinto or how it's implemented that I don't understand, but it seems like it doesn't work like I'd hope.I'm checking with the MTA developer to see whether this behaviour is by design or due to a coding issue.The behaviour that you are observing is due to a legacy restriction from the implementation of sieve filter processing in MS6.3 and below. Sieve filter processing was reworked in MS7.0 and above but the legacy restriction was not removed.
Background on the implementation change here: http://msg.wikidoc.info/index.php/Sieve_Implementation
A bug has been logged for this behaviour to remove the legacy restriction:
bug #6807254 - "Editheader actions don't attach to fileintos in system-level sieves"
With regards to your spamfilter1_string_action sieve rule I should also point out two syntactical errors which aren't causing a problem at this time but are likely to cause the sieve rule to break when future versions of Messaging Server remove "legacy" use of keywords:
require ["spamtest","relational","comparator-i;ascii-numeric","fileinto","addheader"];There is no "addheader" capability string -- this should be "editheader" as per RFC 5293 (http://www.faqs.org/rfcs/rfc5293.html)
addheader "Spam-test: $U";The format for addheader is "addheader" <field-name: string> <value: string>, so this should be:
addheader "Spam-test" "$U";
You can workaround bug #6807254 by moving the fileinto action into a channel level sieve e.g.
<snip option.dat>
spamfilter1_string_action=data:,\
require ["spamtest","relational","comparator-i;ascii-numeric","editheader"];\
spamadjust "$U"; \
addheader "Spam-test" "$U"; \
if spamtest :value "ge" :comparator "i;ascii-numeric" "6" { \
addtag "[SPAM_HI: $U]"; \
} elsif spamtest :value "ge" :comparator "i;ascii-numeric" "4" { \
addtag "[SPAM_LOW: $U]"; \
</snip option.dat>
bash-3.00# cat config/spam.test
require ["spamtest","relational","comparator-i;ascii-numeric","fileinto"];
if spamtest :value "ge" :comparator "i;ascii-numeric" "6" {
        fileinto "spam";
<snip imta.cnf>
! ims-ms
ims-ms defragment subdirs 20 notices 1 7 14 21 28 backoff "pt5m" "pt10m" "pt30m" "pt1h" "pt2h" "pt4h" \
maxjobs 2 pool IMS_POOL fileinto $U+$S@$D filter file:IMTA_TABLE:spam.test destinationspamfilter1
ims-ms-daemon
</snip>Regards,
Shane.