procmail
[Top] [All Lists]

Re: Procmail vs perl script syntax

2006-08-15 06:00:33
At 07:20 2006-08-15 -0500, Jack Stone wrote:
[snip]
I did try to find the author and first sent the questions to the person 
that gave the script to SORBS. No anwser tho.

Yea, I'm thinking if I wrote that, I wouldn't answer my email 
either.  heck, perhaps he's not GETTING his email <g>

Here is the recipe provided that calls the perl script (named origip.pl).
Obviously it is *supposed* to get the correct sending IP and check against 
the SORBS list.

[snip]

Truely idiotic - if they're gonna have you call PERL in the first place to 
do a poor job of parsing the headers, THAT script could have done the 
rblcheck type operation (doing a simple DNS lookup is pretty easy in perl - 
the added code would have been nominal), which would have significantly 
improved the performance of the whole lookup.  If this were being called on 
substantial mail traffic levels, the wasted resources would add up to real 
money.

# First, get the originating IP using the origip.pl script
TCPREMOTEIP=`origip.pl || echo 127.0.0.1`

Three processes: the shell necessary to handle it because there's a pipe, 
the perl process (which must INTERPRET the perlscript), and the echo from a 
shell process.  bleh.  the perl script could at least return the localhost 
value if it didn't find anything.

:0
* ! ? if [ -n "$TCPREMOTEIP" ]; then rblcheck -q -s dnsbl.sorbs.net 
"$TCPREMOTEIP"; fi

More processes the shell to deal with the higher-level test and invoking 
rblcheck.

{
:0 fhw
| formail -A "X-Suspected-Spam: RBL - BLOCKED - dnsbl.sorbs.net"

So that it can turn around and fire up formail to alter the message - this 
part of it of course is inexcapable if you want to modify the message to 
reflect the result (though the functionality could certainly have been 
rolled into the perlscript as well - it's simply adding one line of text on 
the output, and could error out if the host wasn't in the DNSBL).


With sloppy implementations like this, it's no wonder some people get 
aggravated with free software...

---
  Sean B. Straw / Professional Software Engineering

  Procmail disclaimer: <http://www.professional.org/procmail/disclaimer.html>
  Please DO NOT carbon me on list replies.  I'll get my copy from the list.


____________________________________________________________
procmail mailing list   Procmail homepage: http://www.procmail.org/
procmail(_at_)lists(_dot_)RWTH-Aachen(_dot_)DE
http://MailMan.RWTH-Aachen.DE/mailman/listinfo/procmail

<Prev in Thread] Current Thread [Next in Thread>