Hi, 

I finally made the jump from 1.25 to 2.01 and decided this would be
a good time to also apply the qmail-scanner patch from Salvatore.
First off, this is excellent work, I will be sending a donation.  I
did run into a few issues though, mainly with the
settings-per-domain option.  

Here is a snippet from my spd file (using one of my alternate email
addresses - email wrapped this, it's only 2 lines...)

netbits.us:clamdscan,sa,ps'***** Message is Probably SPAM!
*****'2.0''7.0'1 
[EMAIL PROTECTED]:clamdscan,sa,ps'***** Message is Probably SPAM!
*****'2.0''7.0'1''''/usr/local/sites/www.netbits.us/users/josh/.SPAM

First off - I ran into a problem with the generate_spd routine
because of the "!" in the Spam subject.  I've always used this
subject for spam so I didn't want to change it to break client
filters.  In order to generate the .db file I had to change this
line:

    if (/\;|\!/) {

to this:

    if (/\;/) {

Is there a valid reason for not allowing the "!" character in the
file? It seems to work fine with this modification.

Next - when I ran qmail-scanner.pl -p to produce the SPD all went
fine, but running -d I noticed my spamdir was lowercased!  I have
existing client spam dirs as ".SPAM" maildir - so lowercasing is
not an option!  I commented this line:

$domain_settings=tolower("$domain_settings");

in generate_spd again to fix this.  I don't see why you would want
to lowercase something like this?

You will notice that I have full paths to my SPAM dir in the
settings_per_domain.txt file.  Each user already has a SPAM IMAP
dir, I wanted to use sa_quarantene to essentially quarantene the
message to this IMAP dir so they can log into an IMAP client like
webmail to view these messages instead of me having to administer
them from some structure in /var/spool/qscan/quarantene -
unfortunately, the code here was a little too tricky for me to muck
with so I created a symbolic
link 
/var/spool/qscan/quarantene//usr/local/sites/www.netbits.us/users/josh/.SPAM
which points to /usr/local/sites/www.netbits.us/users/josh/.SPAM -
this required updating the -d test in email_quarantine_report to:

if (! -w "$scandir/quarantine/$smaildir")

It would be nice to have absolute path support for sa-spamdir in
future versions - but this is kind of a permissions headache so I
can see what it's not there...

FINALLY - when doing all this I needed to change the code because
my /usr partition is different from /var - the code to uncomment
was:

    #use File::Copy
    #copy ("$scandir/$wmaildir/new/$file_id" etc etc

There was no semi-colon after File::Copy so this caused errors /
temp qmail queue failure until I fixed it.

Thanks, things look great so far - in just over an hour since I
started using patched qmail scanner I've rejected/deleted over 400
spam messages!

Josh

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Qmail-scanner-general mailing list
Qmail-scanner-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qmail-scanner-general

Reply via email to