RFC: filter for Sober.G generated mails

2004-06-11 Thread Hanno Hecker
Hi, this little plugin filters the mails generated by the Sober.G worm. The main reason why it works (worksforme[tm] ;->) is the fsck'd up Message-ID header: qmails Message-IDs always match /^<[EMAIL PROTECTED]>$/ See qmail-1.03/newfield.c, line 26 ff. Maybe some other (spam) mails will be de

Re: RFC: new check_spamhelo

2004-06-13 Thread Hanno Hecker
On Sun, 13 Jun 2004 01:45:15 -0700 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > On Jun 13, 2004, at 1:38 AM, Matt Sergeant wrote: > > > It requires a core patch. I wonder if a new return code might not be > > useful > > here, so you could do: > > > > return CUSTOM, 501, "Syntax error in EHLO a

spaces in (sender's) mail addresses

2004-06-17 Thread Hanno Hecker
Hi, According to RFC 821 an address like <" "@uu-x.de> is a valid address. The code in Qpsmtp/SMTP.pm (sub mail()) just says: my ($from) = ($from_parameter =~ m/^from:\s*(\S+)/i)[0]; A 'MAIL FROM:<" "@uu-x.de>' fails with 450 FQDN required in the envelope sender Changing this line to m

new plugin kavscanner

2004-06-22 Thread Hanno Hecker
Hi, this plugin supports the kasperski virus scanner. The code is a merge of qmail-scanner-queue.pl (see http://qmail-scanner.sourceforge.net/) and the clam_av plugin. The only working (i.e. in production use tested) option is"kavscanner_bin" currently. Please send any updates and/or modifications

$MAXCONNIP for qp-forkserver (was: Re: NOOP handler)

2004-07-05 Thread Hanno Hecker
On Mon, 5 Jul 2004 08:29:37 +0100 Matt Sergeant <[EMAIL PROTECTED]> wrote: > On 4 Jul 2004, at 20:37, Devin Carraway wrote: > > On Sun, Jul 04, 2004 at 07:40:51PM +0100, Mark Powell wrote: > >> Had some dialup joker today, opening 45 smtp connections and > >doing> nothing on them, but a NOOP ever

$MAXLOAD for forkserver

2004-07-10 Thread Hanno Hecker
Hi, this patch adds a MAXLOAD checking for qpsmtpd-forksever, this currently only works in Linux (if you aren't running Linux, send a diff to enable this on your system :)). It also adresses the problem with the broken clients. To disable MAXLOAD and MAXCONNIP checks, use '--max-conn-ip 0' and '-

Re: spaces in (sender's) mail addresses

2004-07-15 Thread Hanno Hecker
On Wed, 14 Jul 2004 17:01:25 -0700 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > On Jun 18, 2004, at 4:01 AM, Peter J. Holzer wrote: > > > On 2004-06-17 12:57:20 -0700, Ask Bjørn Hansen wrote: > >> The solution is to implement an object (Qpsmtpd::MailAddress for > >> example) to handle the SMTP en

srand() for forkserver

2004-07-16 Thread Hanno Hecker
Hi, this adds a call to srand() in the forkserver after the fork(). If it's not called _all_ children of the main daemon will get the same temporary file name when using 'use File::Temp; ($fh, $fname) = tempfile();' in a plugin (like clamav and my kavscanner do). Hanno --- qpsmtpd-forkserver.orig

kavscanner update

2004-07-16 Thread Hanno Hecker
Hi, this is an updated version of my kavscanner plugin. Now the "kav_deny" config file is enabled and known to work :) The kavscanner_bin option now works and the code was rewritten without using any parts from qmail-scanner. Maybe you want to disable the "relay_client" check, as it disables sca

Re: srand() for forkserver

2004-07-17 Thread Hanno Hecker
On Fri, 16 Jul 2004 21:52:58 -0700 Robert Spier <[EMAIL PROTECTED]> wrote: > > +## call srand(), else we will have (e.g.) the same tempfile in > > +## _all_ children > > +## i.e. after 'use File::Temp; ($fh,$name)=tempfile();' in a > > plugin+srand( ($$ ^ $port) ^ (time ^ unpack("C

forkserver: config/plugin options

2004-12-15 Thread Hanno Hecker
Hi, attached are 2 patches. The first one (qp.pm.diff) adds to Qpsmtpd.pm a) a check for $ENV{QPSMTP_CONFIG} in config_dir() and the use of that directory if the wanted config file is in there. All qpsmtpds should be able to use this. b) if load_plugins() gets an argument and it's a dir

Re: forkserver: config/plugin options

2004-12-15 Thread Hanno Hecker
On Wed, 15 Dec 2004 16:07:14 -0800 Robert Spier <[EMAIL PROTECTED]> wrote: > It is probably cleaner to refactor the $0 stuff here instead of just > slamming in another option. (Thus it would be QPSMTPD_ROOT or > something.) It's still got lots of the historical stuff. Also, to [...] > also,

Re: [PATCH] Centralized spool dir config and temporary file/dir handling

2005-02-21 Thread Hanno Hecker
On Mon, 21 Feb 2005 11:41:55 -0500 John Peacock <[EMAIL PROTECTED]> wrote: > The other thing that could be done is to make the return of temp_file > be a subclass of IO::File, so a single hash entry could be both the > filename and the filehandle, but it isn't such a big deal to open the > file

Re: [PATCH] Centralized spool dir config and temporary file/dir

2005-02-22 Thread Hanno Hecker
On Tue, 22 Feb 2005 18:45:12 -0500 Bob <[EMAIL PROTECTED]> wrote: > John Peacock wrote: > > Hanno Hecker wrote: > >- add a 'chmod 0640, $filename;' in plugins/virus/clamav > > before executing clamdscan. > > > Hmmm, allowing another app to tread

Re: ClamAV Rewrite

2005-02-27 Thread Hanno Hecker
On Sun, 27 Feb 2005 08:29:42 -0500 John Peacock <[EMAIL PROTECTED]> wrote: > Since I can't leave well enough alone, I have started to write a new > clamav plugin along the following lines: [...] > 2) directly call clamd on the temporary files that qpsmtpd already > has > (to save writing out a

Re: ClamAV Rewrite

2005-02-28 Thread Hanno Hecker
On Sun, 27 Feb 2005 23:17:59 -0800 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > On Feb 27, 2005, at 10:53 PM, Hanno Hecker wrote: > > > Why you don't write the headers to the temp file when the body > > starts and modify Qpsmtpd::Transaction::body_resetpos() to see

New virus scanner plugin (hbedv)

2005-03-01 Thread Hanno Hecker
dvscanner /path/to/antivir in the plugin config if qpsmtpd, the location defaults to I. =head1 NOTES This plugin started life as a copy of the B plugin. =head1 LICENCE Written by Hanno Hecker E[EMAIL PROTECTED]. The B plugin is published under the same licence as qpsmtpd itself. =cut

Re: New virus scanner plugin (hbedv)

2005-03-01 Thread Hanno Hecker
On Tue, 01 Mar 2005 11:14:17 -0500 John Peacock <[EMAIL PROTECTED]> wrote: > Hanno Hecker wrote: > > Perldoc is included in the plugin. > > When the "full message in spool dir" patch is in CVS I'll update > > this plugin. > > That patch is in CVS

clamdscan (was: Re: New virus scanner plugin (hbedv))

2005-03-01 Thread Hanno Hecker
On Tue, 01 Mar 2005 15:18:42 -0500 John Peacock <[EMAIL PROTECTED]> wrote: > 3) clamdscan - Yet Another ClamAV scanner, this one uses Matt's Clamd > module to talk directly to the clamd daemon (should be faster than the > native clamav plugin). Just a small fix to allow another socket than the def

Re: clamdscan

2005-03-02 Thread Hanno Hecker
On Wed, 02 Mar 2005 09:57:30 -0500 John Peacock <[EMAIL PROTECTED]> wrote: > Hanno Hecker wrote: > > Just a small fix to allow another socket than the default > > /tmp/clamd. > Hmm, I'd actually fix the code differently. The Clamd module doesn't > support

queue via LMTP, just a proof of concept

2005-03-07 Thread Hanno Hecker
Hi all, I just tried to see if it's possible to feed an incoming message directly to the LMTP socket of the cyrus IMAP daemon. This is the result :) Unless all recipients are valid, storing the message will fail...from the view of the sending SMTP daemon... some recipients may have the message in

Re: uniq in perl

2005-06-08 Thread Hanno Hecker
Bob, On Wed, 08 Jun 2005 15:26:29 -0400 Bob <[EMAIL PROTECTED]> wrote: > Nick Leverton wrote: > > There's an even neater way to write uniq, which also preserves the > > order: > #!/usr/bin/perl -w > use strict; # strict and warn fail a lot of anon hash tricks, so... > > sub uniq { > > grep { $

MX pointing to localhost

2005-07-28 Thread Hanno Hecker
Hi *, we had some (smaller) troubles with a spammer sending mails with a resolvable MX, but that MX pointed to 127.0.0.1... after blacklisting these sending IP for a short time I added a check to the require_resolvable_fromhost plugin... Hanno --- require_resolvable_fromhost.orig2005

Re: MX pointing to localhost

2005-07-28 Thread Hanno Hecker
On Thu, 28 Jul 2005 09:38:14 -0400 Keith Ivey <[EMAIL PROTECTED]> wrote: > Hanno Hecker wrote: > > we had some (smaller) troubles with a spammer sending mails with a > > resolvable MX, but that MX pointed to 127.0.0.1... after > > blacklisting these sending IP for a sh

Re: MX pointing to localhost

2005-07-28 Thread Hanno Hecker
On Thu, 28 Jul 2005 11:09:59 -0400 John Peacock <[EMAIL PROTECTED]> wrote: > Hanno Hecker wrote: > > I excluded the check for the RFC 1918 ranges because that server > > delivers to such a range :) If the check for RFC 1918 and 3330 is > > included in this plugin, it s

Re: MX pointing to localhost

2005-07-28 Thread Hanno Hecker
On Thu, 28 Jul 2005 15:15:09 -0400 John Peacock <[EMAIL PROTECTED]> wrote: > I hate to do this to you, but I've been having an offlist conversation > with Roger Walker, who pointed out that some people bind qpsmtpd to > all interfaces (0.0.0.0) and hence localhost might be valid. Rather > than m

rcpt_ldap - look up local users in LDAP

2005-10-05 Thread Hanno Hecker
Hi, while setting up a new system with qpsmtpd and LDAP I took rcpt_ok and auth/auth_ldap_bind, mixed it a bit and this is the result ;-) It uses the same config file as auth_ldap_bind... Currently it's not in production use, some tests have been done to see if it works like intended. While writi

Postfix-queue problem (fixed)

2005-10-16 Thread Hanno Hecker
Hi, I had some trouble setting up virtual_alias_domains and virtual_alias_maps with the queue/postfix plugin. After some testing I found that the queue/postfix plugin does not set any flags for the postfix cleanup. If you want to use something like virtual_alias_domains the CLEANUP_FLAG_MAP_OK is

Re: Postfix-queue problem (fixed)

2005-10-30 Thread Hanno Hecker
Hi Peter, On Sat, 29 Oct 2005 20:02:27 +0200 "Peter J. Holzer" <[EMAIL PROTECTED]> wrote: > > I originally wrote the plugin for postfix 1.x, and I don't remember > > if I didn't set any flags on purpose or if those flags were only > > added with postfix 2.x. > > Postfix 2.2 knows about the follo

Support for RFC 1893 - Enhanced Mail System Status Codes

2005-11-01 Thread Hanno Hecker
Hi *, this is the enhanced version of what I showed Matt on IRC yesterday. It adds the RFC 1893 status codes to the messages which are returned to the sending client. One example is the check_loop plugin. Instead of return DENY, "Too many hops. This message is looping."; it now uses return $se

[PATCH]: pre-/post-connection hooks not working in forkserver

2005-11-12 Thread Hanno Hecker
Hi, this little patch enables the pre-connection and post-connection hooks in qpsmtpd-forkserver. The second attached file is my test plugin. Hanno diff -urN qpsmtpd-0.31-dev/lib/Qpsmtpd/TcpServer.pm qpsmtpd-0.31-dev+pre+post/lib/Qpsmtpd/TcpServer.pm --- qpsmtpd-0.31-dev/lib/Qpsmtpd/Tcp

Re: 0.31rc3

2005-11-14 Thread Hanno Hecker
On Mon, 14 Nov 2005 01:11:21 -0800 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > tar.gz: > http://smtpd.develooper.com/files/qpsmtpd-0.31rc3.tar.gz > > Unless something major comes up then I'll rename the .tar.gz on > Wednesday and we'll call it 0.31. What about ... ... my pre-/post-connec

Re: 0.31rc3

2005-11-14 Thread Hanno Hecker
On Mon, 14 Nov 2005 18:05:28 +0100 Hanno Hecker <[EMAIL PROTECTED]> wrote: > Qpsmtpd::TcpServer::read_input() to qpsmtpd-forkserver (like > attached)? Oops... now the diff is attached :) Hanno diff -ur 0.31rc3/lib/Qpsmtpd/TcpServer.pm 0.31rc3+pre-post-conn/lib/Qpsmtpd/

Re: qpsmtpd-0.31.1-1 rpm

2005-11-30 Thread Hanno Hecker
On Wed, 30 Nov 2005 15:08:43 +0100 "Peter J. Holzer" <[EMAIL PROTECTED]> wrote: > Post-connection hooks don't work. I thought I had included Hanno's > patch, but I haven't. And looking at it again, I think the > post-connection hook is called at the wrong position, so I'll postpone > this until I h

Re: logging incomplete transactions

2005-12-15 Thread Hanno Hecker
On Wed, 14 Dec 2005 17:49:32 -0800 (PST) frank <[EMAIL PROTECTED]> wrote: > Can anyone give me some ideas on how I could implement some code > whereby if the connection is severed prior to an official disconnect > (via QUIT or DENY_DISCONNECT or similar device) that the incident > gets properly

Re: Support for RFC 1893 - Enhanced Mail System Status Codes

2006-01-11 Thread Hanno Hecker
On Wed, 11 Jan 2006 12:15:37 -0500 John Peacock <[EMAIL PROTECTED]> wrote: > Hanno Hecker wrote: > > this is the enhanced version of what I showed Matt on IRC yesterday. > > It adds the RFC 1893 status codes to the messages which are returned > > to the sending client. &

Re: Support for RFC 1893 - Enhanced Mail System Status Codes

2006-01-11 Thread Hanno Hecker
On Wed, 11 Jan 2006 15:19:25 -0500 John Peacock <[EMAIL PROTECTED]> wrote: > > i.e. nothing changes in the core, and every plugin which wants to > > use these status codes has to "use" Qpsmtpd::DSN. > > This seems like duplication of effort. Shouldn't Qpsmtpd::Plugin just > include Qpsmtpd::DS

Re: Starting/Managing qpsmtpd-forkserver

2006-01-15 Thread Hanno Hecker
On Sat, 14 Jan 2006 14:29:29 -0600 Andy Colson <[EMAIL PROTECTED]> wrote: > Along with thoes questions I'd like to add one more. tcpserver can > deny an ip based on some rules in a .cdb file. (tcprules and friends). > I currently use that on my main email box and have a good 50 to 100 > spammers

Re: Starting/Managing qpsmtpd-forkserver

2006-01-16 Thread Hanno Hecker
On Mon, 16 Jan 2006 08:08:37 -0600 Andy Colson <[EMAIL PROTECTED]> wrote: > > In the latest revision of forkserver (601) there's a new plugin, > > which hooks pre-connection: hosts_allow. This allows to have > > hosts/networks not to send mail (and of course exclude some hosts in > > a blocked netw

"invalid" RCPT TOs from relay clients / MAIL FROMs

2006-03-04 Thread Hanno Hecker
Hi *, I've stumbled across a small problem and I'm not sure how to fix it correctly. Whenever a relay client sends an address which is not accepted by Qpsmtpd::Address he never gets an error message. The mail is silently redirected (in our case the admin's address). Some examples: RCPT TO:<[EMAI

abuse & postmaster

2006-03-04 Thread Hanno Hecker
Hi again, why there's a difference in Qpsmtpd::Address and rcpt_ok in accepting mails to "abuse" and "postmaster"? rcpt_ok: $host = $self->qp->config("me") if ($host eq "" && (lc $user eq "postmaster" || lc $user eq "abuse")); Qpsmtpd::Address: # bare postmaster

Re: "invalid" RCPT TOs from relay clients / MAIL FROMs

2006-03-06 Thread Hanno Hecker
On Sat, 4 Mar 2006 19:35:56 +0100 "Peter J. Holzer" <[EMAIL PROTECTED]> wrote: > I think this got lost during the "why do we need both new and parse?" > discussion a few months ago. That would explain why it worked with > 0.28. > > the attached patch should fix it. If the missing ";" on the previo

Re: A different angle on greylisting

2006-03-12 Thread Hanno Hecker
On Mon, 13 Mar 2006 13:06:15 +1100 Gavin Carr <[EMAIL PROTECTED]> wrote: > > I am thinking, for example, if spamassassin scores 3 or higher to > > softfail and add to the greylist database. Chances are we'll never > > see it again. If its from a real site then it should just be > > delayed t

Re: block_subject plugin

2006-03-14 Thread Hanno Hecker
On Tue, 14 Mar 2006 09:14:09 -0700 Bryan Scott <[EMAIL PROTECTED]> wrote: > I disabled warnings to avoid filling my logs with pattern matching > errors. If others have suggestions as to the safe perl mechanisms for > proper pattern matching from values in a file, let me know. Safe depends on trus

wiki: hook docs updated

2006-03-14 Thread Hanno Hecker
Hi all, I've updated the plugin hooks page of the wiki: http://wiki.qpsmtpd.org/plugin_hooks This should be now more complete than the README.plugins Please review and edit any mistakes :) Hanno

Re: [perl #38747] RCPT TO addresses with spaces are not correctly parsed

2006-03-17 Thread Hanno Hecker
On Thu, 16 Mar 2006 23:24:38 -0800 Robert Spier <[EMAIL PROTECTED]> wrote: > > + my ($rcpt) = ($rcpt_parameter =~ m/^to:\s*(<[^>]*>)/i)[0]; > > + # support addresses without <> ... maybe we shouldn't? > > + ($rcpt) = "<" . ($rcpt_parameter =~ m/^to:\s*(\S+)/i)[0] . ">" > > + unless $rcpt;

Re: [perl #38747] RCPT TO addresses with spaces are not correctly parsed

2006-03-18 Thread Hanno Hecker
On Fri, 17 Mar 2006 14:07:35 -0800 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > On Mar 16, 2006, at 10:38 PM, Hanno Hecker (via RT) wrote: > > I'd like to see a patch that makes a better API for reading the > command and the command parameters. The default can still

Re: problems getting qpsmtpd to work with postfix alias domains

2006-03-22 Thread Hanno Hecker
On Thu, 23 Mar 2006 16:58:31 +1100 "john habermann" <[EMAIL PROTECTED]> wrote: > I have been trying to test qpsmtpd and have it set up on a debian > stable server passing messages on to postfix. It works fine except > when I try and get it to work with postfix virtual alias domains. I [...] > If i

Re: --max-from-ip doesn't work

2006-03-25 Thread Hanno Hecker
On Sat, 25 Mar 2006 15:45:07 + (UTC) [EMAIL PROTECTED] wrote: > I've got my qpsmtpd-forkserver (0.32 updated with some > branch/3.x > for TLS/AUTH fixes) run with "--max-from-ip 5" and I just counted 10 > connections from a single IP. >From config.sample/plugins: # The hosts_all

Re: [perl #38747] RCPT TO addresses with spaces are not correctly parsed

2006-04-02 Thread Hanno Hecker
On Fri, 17 Mar 2006 14:07:35 -0800 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > I'd like to see a patch that makes a better API for reading the > command and the command parameters. The default can still be having > it space separated and provided in @_, but the hacks in mail() and > rcpt()

Re: Share Greylisting DB Across Multiple Machines

2006-04-11 Thread Hanno Hecker
Hi Max, On Tue, 11 Apr 2006 09:44:34 +1000 Gavin Carr <[EMAIL PROTECTED]> wrote: > On Mon, Apr 10, 2006 at 12:39:27PM -0700, Max Clark wrote: > > I need to share my greylisting data across multiple machines - does > > a patch exist to read/write this data to a mysql database? I'm > > assuming this

Re: Case significance

2006-04-24 Thread Hanno Hecker
On Mon, 24 Apr 2006 18:05:31 +1000 Andrew Pam <[EMAIL PROTECTED]> wrote: > Apologies if this has been discussed before. I have a number of > clients with mailboxes on my server who are complaining that they are > losing important incoming emails. On further investigation, this > turns out to be

Re: ignore load balancer "alive" checks

2006-07-20 Thread Hanno Hecker
Hi Jared, On Thu, 20 Jul 2006 17:00:31 + Jared P Bostic <[EMAIL PROTECTED]> wrote: > I'm setting up 0.32 behind a load balancer and noticed > that the TCP service checks that it sends are really > annoying to sift through in the log files. I would like to > use fairly verbose logging for test

[PATCH] $transaction->body_fh()

2006-07-23 Thread Hanno Hecker
Hi all, this patch adds a body_fh() call to Qpsmtpd::Transaction. It returns a file handle to the temporary file with the message (or undef if the file is not open yet). The first diff is a minimum, i.e. just adds the sub and pod, the second replaces (nearly) all $self->{_body_file} with $self->b

new plugin 'chunking' / plugin for the CHUNKING SMTP extension (RFC 1830/3030)

2006-07-23 Thread Hanno Hecker
Hi all, the chunking plugin adds support for the CHUNKING SMTP extension (RFC 1830/3030). This has not been tested in the wild, because I haven't found any free MTA which supports this :-( Testing and debugging was done with the attached send-chunks.pl. This plugin requires the transaction-body

Re: Qpsmtpd::DSN message format

2006-08-06 Thread Hanno Hecker
Hi Peter, On Sun, 6 Aug 2006 11:41:21 +0200 "Peter J. Holzer" <[EMAIL PROTECTED]> wrote: > While changing the spamassassin plugin to allow per-user > configuration, I noticed that Qpsmtpd::DSN produces messages like: > > 552 spamassassin score 15.6/10 (#5.7.1) > > According to RFC 2034 the exte

Re: Suggestions, advice for Qpsmtpd::Transaction->body_front_write

2006-08-23 Thread Hanno Hecker
Hi Fred, On Wed, 23 Aug 2006 01:53:11 -0700 Fred Moyer <[EMAIL PROTECTED]> wrote: > David Nicol wrote: > Here's the revised version, a bit shorter: [...] > # write the data and recalculate length > $self->{_body_file}->print(ref $data eq "SCALAR" ? > $$data . $self->body_as_strin

Re: Debris left in $spool_dir under forkserver

2006-09-23 Thread Hanno Hecker
Hi Charlie, On Sat, 23 Sep 2006 10:55:49 -0400 (EDT) Charlie Brady <[EMAIL PROTECTED]> wrote: > I am still convinced that the best solution to this problem is to use > anonymous files in the spool directory. Unlink the files as soon as > they are created, and manipulate the content only via an

Re: postfix/cleanup status returns

2006-10-26 Thread Hanno Hecker
On Thu, 26 Oct 2006 09:31:37 -0400 John Peacock <[EMAIL PROTECTED]> wrote: > David Muir Sharnoff wrote: > > +if ($status) { > > +# this section needs to be kept in sync with the > > cleanup_stat_map +# array found in Postfix source file > > src/global/cleanup_strerror.c +# which in

[PATCH] Postfix constants (was Re: postfix/cleanup status returns)

2006-10-28 Thread Hanno Hecker
On Thu, 26 Oct 2006 09:31:37 -0400 John Peacock <[EMAIL PROTECTED]> wrote: > David Muir Sharnoff wrote: > > +if ($status) { > > +# this section needs to be kept in sync with the > > cleanup_stat_map +# array found in Postfix source file > > src/global/cleanup_strerror.c +# which in

Re: [PATCH] Postfix constants (was Re: postfix/cleanup status returns)

2006-10-31 Thread Hanno Hecker
On Sat, 28 Oct 2006 12:40:05 -0700 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > It'd be good if we could have some postfix compatibility notes > documented; either in Qpsmtpd::Postfix or in the plugin. With some changes to the code... the plugins/queue/postfix-queue now contains a pod section wi

[PATCH] and new plugin "skip_plugins"

2006-10-31 Thread Hanno Hecker
Hi *, the "skip_plugins" plugin lets you skip selected plugins for some hosts. This is similar to the whitelist plugins, but - the config is done in one config file for all plugins - no plugin needs to have the whitelist tests - each plugin can be disabled (or re-enabled) on the fly by some

Re: qpsmtpd and mail recipients

2006-11-02 Thread Hanno Hecker
On Thu, 02 Nov 2006 18:11:30 + R J Thompson <[EMAIL PROTECTED]> wrote: > Would it be possible to change the recipient of an email if a spam or > virus is found to send to an virus@ or spam@ account, I thought about yep, it's possible... see perldoc Qpsmtpd::Transaction -> recipients( )

Plugin disabling API (was: Re: [PATCH] and new plugin "skip_plugins")

2006-11-05 Thread Hanno Hecker
On Tue, 31 Oct 2006 19:56:11 -0800 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > On Oct 31, 2006, at 6:38 PM, John Peacock wrote: > > > The issue is that this plugin requires a patch to the core > > to actually implement the skip part, which might be fine, but there > > might also > > be a more e

Re: Mails with odd dates bypassing check_basicheaders with day count.

2006-11-14 Thread Hanno Hecker
Hi Robin, On Mon, 13 Nov 2006 22:42:24 -0800 "Robin H. Johnson" <[EMAIL PROTECTED]> wrote: > I've recently started seeing a lot of spam with insane dates bypassing > teh check_basicheaders day count. > > A Date line from two of the emails: > Tue, 19 Jan 2038 11:14:07 +0800 > Wed, 23 M

Re: Apache::Qpsmtpd troubles

2006-11-17 Thread Hanno Hecker
On Fri, 17 Nov 2006 11:22:29 -0500 John Peacock <[EMAIL PROTECTED]> wrote: > Just to make clear - it's not specific to tls, but fails the same way > for any plugin. I earlier got the error trying to load any of the > logging plugins. You're using svn rev 671, 672 or 673? If yes: use 674 or set

Re: Can we make forkserver ignore IP's...

2006-11-27 Thread Hanno Hecker
On Mon, 27 Nov 2006 08:43:43 -0700 Bryan Scott <[EMAIL PROTECTED]> wrote: > The first thought I had was if we could make the forkserver code > perform similarly. It wouldn't be able to block just based on the > SYN packets, but it could immediately close the connection (no > banner, no error me

[PATCH] $transaction->body_fh, $transaction->body_length

2006-12-05 Thread Hanno Hecker
Hi, this patch adds two small functions to Qpsmtpd::Transaction. - body_fh() returns the file handle for the temp file (if spooled to disk) or undef (if not spooled to disk). - body_length() - returns the current length of the body of the email (including the empty line between header an

Re: [PATCH] $transaction->body_fh, $transaction->body_length

2006-12-05 Thread Hanno Hecker
On Tue, 5 Dec 2006 13:43:03 -0800 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > On Dec 5, 2006, at 9:29, Hanno Hecker wrote: > > - body_length() - returns the current length of the body of the > > email > > (including the empty line between header and body), no mat

Fw: Re: [PATCH] $transaction->body_fh, $transaction->body_length

2006-12-08 Thread Hanno Hecker
...and now also to list :) Begin forwarded message: Date: Fri, 8 Dec 2006 19:17:07 +0100 From: Hanno Hecker <[EMAIL PROTECTED]> To: Ask Bjørn Hansen <[EMAIL PROTECTED]> Subject: Re: [PATCH] $transaction->body_fh, $transaction->body_length On Fri, 8 Dec 2006 01:56:15 -080

Re: [PATCH] $transaction->body_fh, $transaction->body_length

2006-12-16 Thread Hanno Hecker
Hi all, On Fri, 8 Dec 2006 01:56:15 -0800 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > >> Maybe we should rename body_size to message_(length|size)?(Of > >> course keep body_size working, but deprecating it...) > > > > I think I'd prefer data_size, to correspond to the SMTP command. > > I

Re: Many miscellaneous 0.32 patches

2006-12-25 Thread Hanno Hecker
Hi Michael, On Sun, 24 Dec 2006 12:03:04 -0500 "Michael C. Toren" <[EMAIL PROTECTED]> wrote: > I recently transitioned mail.netisland.net from 0.28-tcpserver to > 0.32-forkserver, and in doing so ended up making a number of small > changes. The attached diff does the following: > > o lib/Qpsm

Re: [svn:qpsmtpd] r699 - branches/0.3x/lib/Qpsmtpd

2006-12-27 Thread Hanno Hecker
On Mon, 25 Dec 2006 23:17:53 +0100 Ask Bjørn Hansen <[EMAIL PROTECTED]> wrote: > >> Log: > >> some fixes from Michael C. Toren > > > > Hanno, > > > > Please be more specific with change descriptions... it makes > > figuring out what happened much easier. Ok, remembering for the next commits.

Re: which hook provides sender and rcpt at once?

2006-12-30 Thread Hanno Hecker
Hi Joe, On Sat, 30 Dec 2006 05:11:00 +0100 Joe Knall <[EMAIL PROTECTED]> wrote: > I'm new to qpsmtpd and want a plugin for userdefined black/whitelisting > of senders. The plugin should look up in a database if the recipient > has listed the current sender and react appropriate (allow, drop, >

Re: which hook provides sender and rcpt at once?

2006-12-30 Thread Hanno Hecker
On Sun, 31 Dec 2006 01:40:47 +0100 Joe Knall <[EMAIL PROTECTED]> wrote: > On Sat, 2006-12-2006 09:17 Hanno Hecker wrote: > > sub hook_rcpt { > >     my ($self, $transaction, $recipient, %param) = @_; > >     my $sender = $transaction->sender; > > > &

Disabling and re-enabling loaded plugins

2006-12-31 Thread Hanno Hecker
Hi *, I just committed my patch to disable and re-enable loaded plugins. An example plugin "skip_plugins" is included. See perldoc Qpsmtpd::Plugin and the "skip_plugins" plugin to find out how to use this in detail. The plugins can be disabled for the current connection only (not for just a trans

Re: which hook provides sender and rcpt at once?

2006-12-31 Thread Hanno Hecker
On Sun, 31 Dec 2006 06:52:25 -0500 Guy Hulbert <[EMAIL PROTECTED]> wrote: > On Sun, 2006-31-12 at 08:59 +0100, Hanno Hecker wrote: > > The best thing[tm] would be a uniform Qpsmtpd::DBI module, which > > handles all flavours of qpsmtpd. I can do this for -forkserver and >

Re: [svn:qpsmtpd] r700 - in branches/0.3x: . config.sample lib lib/Qpsmtpd plugins

2007-01-07 Thread Hanno Hecker
On Sat, 06 Jan 2007 23:40:27 -0800 Robert Spier <[EMAIL PROTECTED]> wrote: > > Please see my "peers" plugin posted a few months ago > > (http://www.nntp.perl.org/group/perl.qpsmtpd/4475). It allows > > different (arbitrary) sets of plugins to be hooked for different > > ranges of connecting IP add

Re: warnings in spamassassin plugin

2007-01-08 Thread Hanno Hecker
Hi, On Mon, 08 Jan 2007 12:15:48 + [EMAIL PROTECTED] wrote: > I'm getting some unitialized var warnings in the spamassassin plugin. > Nothing to worry about I presume. Just a note to the maintainer. [...] > 2007-01-08 12:12:18.287066500 15607 spamassassin plugin: check_spam: > spamd: Content-

Re: handle multiple recipients differently?

2007-01-13 Thread Hanno Hecker
On Sat, 13 Jan 2007 20:57:05 +0100 Joe Knall <[EMAIL PROTECTED]> wrote: > As far as I understand a mail with two recipients triggers hook_rcpt > twice. Problem: hook_rcpt returns only one return value. Yes, one for each recipient. > Imagine a mail like this: > echo test | mail -r [EMAIL PROTECTE

Re: New Plugin Documentation

2007-01-22 Thread Hanno Hecker
Hi Guy, On Mon, 22 Jan 2007 12:34:44 -0500 Guy Hulbert <[EMAIL PROTECTED]> wrote: > James Turnbull, Michael Holtz, and I continue to discuss this off-list > but up until now they have done all the work ... James and I are putting > new documentation in the playground for now but please come in and

Re: smtp connections never timeout

2007-01-23 Thread Hanno Hecker
On Tue, 23 Jan 2007 11:31:51 -0600 Oscar Retana <[EMAIL PROTECTED]> wrote: > BUT yesterday I couldn't find any culprit plug-in. In fact, I > disabled ALL plug-ins. Basically my qpsmtpd/config/plugins file was > left to this (for testing): > > rcpt_ok > queue/qmail-queue > > Nothing... Then I

Re: new plugin: db_whitelist_soft WAS writing plugins (whitelist with database)

2007-02-06 Thread Hanno Hecker
On Tue, 06 Feb 2007 17:22:55 -0700 JT Moree <[EMAIL PROTECTED]> wrote: > I have hacked the whitelist_soft plugin into a new one that uses > database backends. I need some help on database issues. > > The plugin works fine for the first run but then the database connection > goes away. (MySQL) I

Re: cleanup hook under xinetd?

2007-03-01 Thread Hanno Hecker
On Wed, 28 Feb 2007 20:52:21 +0100 Hans Salvisberg <[EMAIL PROTECTED]> wrote: > Is there a hook that is always fired at the end of a connection under > xinetd? > > If qpsmtpd disconnects, then hook_disconnect() and > hook_post_connection() are fired. But if the other side disconnects > without

Re: cleanup hook under xinetd?

2007-03-03 Thread Hanno Hecker
On Sat, 03 Mar 2007 08:33:16 +0100 Hans Salvisberg <[EMAIL PROTECTED]> wrote: > Hanno Hecker wrote: > > On Wed, 28 Feb 2007 20:52:21 +0100 > > Hans Salvisberg <[EMAIL PROTECTED]> wrote: > >> If qpsmtpd disconnects, then hook_disconnect() and > >>

Re: Debian Package Question

2007-05-10 Thread Hanno Hecker
On Thu, 10 May 2007 09:52:14 -0400 Guy Hulbert <[EMAIL PROTECTED]> wrote: > Why does the debian package depend on adduser? because it adds the 'qpsmtpd' user if it doesn't exist? Hanno

Re: shutdown hook for plugins?

2007-07-31 Thread Hanno Hecker
Hi Matthew, On Mon, 30 Jul 2007 20:09:49 -0400 Matthew Harrell <[EMAIL PROTECTED]> wrote: > When writing a plugin I can do initialization stuff in the register method. > Is there a method I can use that will be called when qpsmtpd is shutting > down? There's an RRD plugin I'm fiddling with and I

Re: shutdown hook for plugins?

2007-07-31 Thread Hanno Hecker
Hi Matthew, On Tue, 31 Jul 2007 07:13:37 -0400 Matthew Harrell <[EMAIL PROTECTED]> wrote: > : Depending on which flavour of qpsmtpd you're running, this (register()) > : is probably not be the right place for opening connections. The connection > : would be shared by all sub processes of qpsmtpd a

Plugin documentation

2007-08-06 Thread Hanno Hecker
Hi *, before submitting such a large chunk of text (35k), I'd like to have some feedback. Spelling, additions, comments, fixes (not only those marked "FIXME" ;->), ... Oh... this is the "preview": http://ankh-morp.org/~vetinari/qpsmtpd/plugins.pod Hanno

Queue speed

2007-08-10 Thread Hanno Hecker
Hi, while playing around with large binary mails I found that a simple change to the queue/* plugins can speed up queueing quite a bit. With the default $transaction->body_getline loop it takes ~0.5 seconds (on my test system) to queue an 18MB Mail to a maildir. With a simple diff it drops down to

Re: bug report: data_post and DENY{,SOFT}_DISCONNECT

2007-08-15 Thread Hanno Hecker
On Wed, 15 Aug 2007 06:33:15 + (UTC) Brad Fitzpatrick <[EMAIL PROTECTED]> wrote: > To whom it may concern (and can commit :)), > > README.plugins documents, for the data_post hook: > >DENY_DISCONNECT & DENYSOFT_DISCONNECT - as above but with disconnect > > But in Qpsmtpd/SMTP.pm, it does

Re: [svn:qpsmtpd] r772 - trunk/lib/Qpsmtpd/SMTP

2007-08-22 Thread Hanno Hecker
On Tue, 21 Aug 2007 21:00:19 -0700 Robert Spier <[EMAIL PROTECTED]> wrote: > This CL appears to remove the functionality of processing the hook's > return value. Am I misreading it? Yes, it removes it from this part. The return values are checked in Qpsmtpd::SMTP::unrecognized_command_respond(),

Transaction IDs (was: Re: helper scripts)

2007-08-23 Thread Hanno Hecker
On Wed, 22 Aug 2007 12:48:09 -0700 JT Moree <[EMAIL PROTECTED]> wrote: > Sydney Bogaert wrote: > > You assume here that the process numbers are different for each message. > > While it will work for forkserver and tcpserver, that is not the case > > with prefork or Apache (also preforking). > > Is

Re: isa_plugin problem

2007-08-30 Thread Hanno Hecker
On Fri, 31 Aug 2007 12:29:00 +1000 Gavin Carr <[EMAIL PROTECTED]> wrote: [...] > I think I've seen something similar, which turned out to be a bug > in the 0.40 isa_plugin code not handling multiple plugin_dirs > properly. Patch attached. Not sure if it's your problem but it > smells similar. Tha

Re: Bugs in qpsmtpd-prefork + some features + patches

2007-08-31 Thread Hanno Hecker
On Thu, 30 Aug 2007 08:50:40 +0200 Stefan Priebe <[EMAIL PROTECTED]> wrote: > So here are some problems and the solutions ( the patches may not apply > cleanly to the actual 0.40 version): > > > 1.) You get some errors with the auth mechanism if an old process has > used auth the value seems no

Re: Bugs in qpsmtpd-prefork + some features + patches

2007-08-31 Thread Hanno Hecker
Hi Stefan, [Cc: to ML again] On Fri, 31 Aug 2007 13:58:07 +0200 Stefan Priebe <[EMAIL PROTECTED]> wrote: > To 1.) i'm not shure but it works perfectly with only deleting the > _auth. The main question is - what gives us MORE performance (delete the > values or create a new instance). The clean

Re: Bugs in qpsmtpd-prefork + some features + patches

2007-09-01 Thread Hanno Hecker
On Fri, 31 Aug 2007 13:58:26 +0200 Stefan Priebe <[EMAIL PROTECTED]> wrote: > To 1.) i'm not shure but it works perfectly with only deleting the > _auth. The main question is - what gives us MORE performance (delete the > values or create a new instance). The clean solution is, to create a new >

Re: Bugs in qpsmtpd-prefork + some features + patches

2007-09-02 Thread Hanno Hecker
On Fri, 31 Aug 2007 13:58:07 +0200 Stefan Priebe <[EMAIL PROTECTED]> wrote: > To 4.) perhaps the new instance also solves this problem. I don't > remember the failure - i only know that it was TLS related. And the > connection fails after TLS command. It does not solve it, it seems to be a proble

Re: Transaction IDs (was: Re: helper scripts)

2007-09-02 Thread Hanno Hecker
On Sun, 2 Sep 2007 18:48:00 +0200 "Peter J. Holzer" <[EMAIL PROTECTED]> wrote: > > + $self->respond(250, $from->format . ", sender OK - your transaction > > id is ".$self->transaction->id); > I like that, although I suspect that most clients will just discard it. But some clients show the t

Re: Bugs in qpsmtpd-prefork + some features + patches

2007-09-03 Thread Hanno Hecker
;reset_transaction', the final hook_reset_transaction() will not be called if you exit in hook_disconnect(). The only way not to call the post-connection hook in -prefork (and -forkserver) is to kill the process. > Hanno Hecker schrieb: > > For now it seems like the only solution is to exit the process

  1   2   3   >