Re: [Qmail-scanner-general]Missing SA headers in st-patched QS.

[Salvatore T. <to...@pu...>]

At 19:33 +0900 27-07-2004, michihito matsubara wrote:
>Hi lists.
>
>I'm comparing three variant of qmail-scanner w/heavily patched qmail.
>
>#1 qmail-scanner-1.22        (original)
>#2 qmail-scanner-1.22-st     (q-s-1.22st-20040606.patch.gz)
>#3 qmail-scanner-1.22-st-qms (qmail-scanner-1.22-st-qms-20040530.patch)
>
>Every QS works fine except 3 points below.
>
>1)
>I wonder that both patched QS doesn't have SPAM_REPORT info any more,
>even original has it. Like below,

>Patched QS have set $sa_alt='1' and $sa_debug='1'.
>I found articles from list archive about spamc option
>to switch fast_spamassassin to verbose_spamassassin by removing '-c'.
>so i've removed '-c' option from $spamc_options, but doesn't help.

>I know st-patch is a heavily modifying against QS,
>but I don't think this is a normal behavier.

Well, this is a normal behavior.

'sa_alt' runs in 'fast_spamassissin' mode and it ignores the $spamc_options.

sa_debug gets the SPAM_REPORT and write it to the logs and to the 
report in the bottom of quarantine message, and it works only if you 
enable sa_alt.

I worked a bit to include the SPAM_REPORT in the message headers, but 
I found some problems with the regex to extract the report and 
sometimes there is some garbage in the report. I don't mind if I have 
some garbage in the log but I don't think I can add garbage to header 
that my users could see.

If you want to include the report disable sa_alt and to change spamc_options:


my $sa_alt='0';
.........
# st: fast_spamassassin options=" -c -f " / verbose_spamassassion 
options=" -f "
my $spamc_options=' -f';



Then, you have to set spamassassin local.cf to add the report to 
headers and eventualy modify the subject since qmail-scanner doesn't 
modify the subject when it works in verbose_spamassassi mode.

The report won't be written to qmail-queue.log nor to the botton of 
quarantine messages nor to the admin notifies.

I will fix the bug in the sub-uudecode.pl in the next version. Thanks for that.

Salvatore