Re: Tokyo Cabinet as a BayesStore

2009-04-23 Thread Matt Sergeant
On Thu, 23 Apr 2009 10:23:30 -0400, Matt Sergeant wrote: On Thu, 23 Apr 2009 10:32:13 +, Justin Mason wrote: http://anyall.org/blog/2009/04/performance-comparison-keyvalue-stores-for-language-model-counts/ highlight: a Tokyo Cabinet hashtable performed at 1400 ops/sec compared

Re: ANN - DBD::SQLite version 1.20

2009-04-07 Thread Matt Sergeant
Congrats guys - glad this finally came about. On Tue, 7 Apr 2009, Darren Duncan wrote: All, I am pleased to announce that DBD::SQLite (Self Contained RDBMS in a DBI Driver) version 1.20 has been released on CPAN. http://search.cpan.org/dist/DBD-SQLite/ This follows on the heels of 10

Re: perltidy

2009-04-03 Thread Matt Sergeant
On Fri, 3 Apr 2009 08:38:59 -0500, Jared Johnson wrote: first of all, kudos on the frequent releasing :) I've attached a suggested patch to .perltidyrc. I've been playing around with perltidy'ing all QP code and some results I don't like. This doesn't fix all the things that I don't like,

Re: Who wants to play holy war over -ce ?

2009-04-03 Thread Matt Sergeant
On Fri, 3 Apr 2009, David Nicol wrote: On Fri, Apr 3, 2009 at 12:36 PM, Matt Sergeant m...@sergeant.org wrote:        -ce,   --cuddled-else You're the first perl programmer I know who likes these :-) I don't. And I'm fairly sure the rest of us probably don't either. I use them

Re: RFC: developing DBD::SQLite

2009-03-27 Thread Matt Sergeant
On Fri, 27 Mar 2009, Greg Sabino Mullane wrote: I have tried emailing Matt several times without response already. Did you cc modu...@perl.org? What did they say? They've been helpful with me in the past in tracking module owners down. I very rarely read mailing list mail these days - my

Re: [sqlite] speeding up row by row lookup in a large db

2009-03-23 Thread Matt Sergeant
On Sat, 21 Mar 2009 23:42:30 +0100, Stefan Evert wrote: > On 21 Mar 2009, at 15:31, P Kishor wrote: > >> I did some benchmarking with the above schema using Perl DBI, and I >> get about 30 transactions per second as long as I returning the data >> to memory. > > Even for Perl/DBI, that seems

Re: [sqlite] speeding up row by row lookup in a large db

2009-03-23 Thread Matt Sergeant
On Sat, 21 Mar 2009 23:42:30 +0100, Stefan Evert wrote: On 21 Mar 2009, at 15:31, P Kishor wrote: I did some benchmarking with the above schema using Perl DBI, and I get about 30 transactions per second as long as I returning the data to memory. Even for Perl/DBI, that seems pretty slow.

Re: End of headers hook

2009-02-10 Thread Matt Sergeant
Also note this won't work with -async properly. I'll have a look how it can be made to work asynchronously (you need to follow the respond style in the rest of the code). On Mon, 09 Feb 2009 23:43:05 -0800, Robert Spier wrote: Committed as

Re: qpsmtpd-async under AnyEvent

2009-02-03 Thread Matt Sergeant
On Tue, 3 Feb 2009, Pedro Melo wrote: Hi, I have a AnyEvent-based project that requires a SMTP server. I was considering either write a AnyEvent::Impl::Danga::Socket so that I can run qpsmtpd directly. Has anybody played with something like this here? I don't think so... Though I have a

Re: Domain Alias in Qpsmtpd

2009-01-29 Thread Matt Sergeant
On Wed, 28 Jan 2009 16:23:01 +0100, Julien wrote: The config file (/var/qmail/control/aliasdomains) contain one remplacment per line, each remplacment consist of 'olddomain' and 'newdomain' seperate by a ':' like : test.domain.com:domain.com So address jul...@test.domain.com sould be

Re: Domain Alias in Qpsmtpd

2009-01-28 Thread Matt Sergeant
On Wed, 28 Jan 2009 15:13:31 +0100, Julien wrote: Without any success, I still got/get the original recipient. What version of qpsmtpd? I think changing these values was only added in SVN. Matt.

Re: Have at it

2009-01-26 Thread Matt Sergeant
On Sat, 24 Jan 2009 22:38:40 +, Simon Cozens wrote: http://www.dev411.com/blog/2009/01/14/perl-5-for-the-future-the-enlightened-perl-organization Umm, this: The goal is to modernize Perl 5 and make it competitive with new developments in programming languages, given that it's unknown

Re: File Permissions bug?

2009-01-25 Thread Matt Sergeant
On Sat, 24 Jan 2009 10:37:46 -0500 (EST), Charlie Brady wrote: Any time you spend doing useless system calls isn't available for spamassassin or clamav. Run strace against qpsmtpd (at least forkserver variant) some time and you'll see how often config files are opened. Well of course.

File Permissions bug?

2009-01-21 Thread Matt Sergeant
Should we fix qpsmtpd config loading to check for file permissions as described in: http://use.perl.org/~Alias/journal/38319 ? Matt.

Re: indentation, line length, trailing whitespace

2009-01-14 Thread Matt Sergeant
On Wed, 14 Jan 2009 09:10:14 -0600, Jared Johnson wrote: Hi, Is there any interest in patches to conform current QP code to line length and whitespace standards? 1. I notice that most of the codebase uses 4-space indents, but I have seen some areas with 2-space indents and tab

Re: [PATCH] Make remote_host available in hook_pre_connection

2009-01-12 Thread Matt Sergeant
On Mon, 12 Jan 2009, Jared Johnson wrote: I propose something like the following in the connection package sub remote_hostname{ my $self = shift; $self-{_remote_hostname} ||= Net::DNS::get_name($self-{_remote_ip}) } This seems reasonable to me on principal. Except

Re: Christmas release?

2008-12-23 Thread Matt Sergeant
On Mon, 22 Dec 2008 11:58:23 +0100, Kaare Rasmussen wrote: On Dec 19, 2008, at 18:56, Matt Sergeant m...@sergeant.org wrote: Shall we do a release for xmas? It's been forever... On the topic of releasing, why isn't qpsmtpd released to CPAN? Because we never really got it easily installable

Re: general question about ClamAV plugin

2008-10-01 Thread Matt Sergeant
On Tue, 30 Sep 2008 17:03:36 -0700 (PDT), [EMAIL PROTECTED] wrote: When using the ClamAV plugin i realize that this is not the entire ClamAV application but how do I then keep the virus definitions for the plugin up to date? do I need to actually install the ClamAV application to use freshclam

Re: Latest SMTP.pm causes fatal errors

2008-09-30 Thread Matt Sergeant
On Tue, 30 Sep 2008 08:34:05 +0200, Hanno Hecker wrote: I'm confused why the error is caused - anyone better with perl than me can help out? Same confusion here... and I cannot reproduce it. $ perl -v | grep 'This is perl' This is perl, v5.8.8 built for i486-linux-gnu-thread-multi

Re: DNSBL and answer id check missing

2008-09-27 Thread Matt Sergeant
On Sat, 27 Sep 2008 13:56:58 +0200, Diego d'Ambra wrote: To me it seems that plugin DNSBL is using Net::DNS bgsend/bgread, but is not checking the id of the reply received. If true this means that an attacker can white- or blacklist any email by sending fake dns replies (only randomisation

Re: DNSBL and answer id check missing

2008-09-27 Thread Matt Sergeant
On Sat, 27 Sep 2008 20:09:37 -0400, Chris Lewis wrote: I've extended the async dnsbl plugin to do scoring. It occured to me a few days ago that DNSBLs with negative scores (DNSWLs) should be treated as a hit if they get a timeout or other failure. This has prompted me to comment about

RE: FIXED -- high CPU on lost processes with forkserver

2008-09-26 Thread Matt Sergeant
catch SIGPIPE but don't call the previously installed signal handler. So only one of them gets called (whichever is registered last) and the other one loses. So before patching the qp core in the respond method (Matt Sergeant commented: But I removed it because then alarm() features VERY

EnemiesList plugin

2008-09-26 Thread Matt Sergeant
People who follow the SVN commits list will have seen that I posted an enemies_list plugin in contrib. EnemiesList is a database of regular expressions that map to dynamic-looking hosts. It's non-free, which is why I've posted it in the contrib directory. In the EL contrib directory (on their

Re: $transaction-body_filename;

2008-09-15 Thread Matt Sergeant
On Mon, 15 Sep 2008 16:40:24 -0400, Chris Lewis wrote: According to the documentation, when you call $transaction-body_filename, you get a temporary file name that points at a file that contains the message. If you examine body_filename, it has no headers. The clamdscan plugin uses

Re: $transaction-body_filename;

2008-09-15 Thread Matt Sergeant
On Mon, 15 Sep 2008 17:39:33 -0400, Chris Lewis wrote: Matt Sergeant wrote: On Mon, 15 Sep 2008 16:40:24 -0400, Chris Lewis wrote: According to the documentation, when you call $transaction-body_filename, you get a temporary file name that points at a file that contains the message. If you

Re: $transaction-body_filename;

2008-09-15 Thread Matt Sergeant
On Mon, 15 Sep 2008 19:30:11 -0400, Matt Sergeant wrote: Would it be worth considering have a data_filename() call, that does exactly the same thing as body_filename, but includes the headers too? Then we can fix the clamdscan plugin without breaking anything else. I thought about

Re: $transaction-body_filename;

2008-09-15 Thread Matt Sergeant
On Mon, 15 Sep 2008 17:35:31 -0700, Ask Bjørn Hansen wrote: On Sep 15, 2008, at 13:40, Chris Lewis wrote: According to the documentation, when you call $transaction-body_filename, you get a temporary file name that points at a file that contains the message. If you examine body_filename,

Re: qpsmtpd-prefork - anyone using it and are patches accepted?

2008-08-04 Thread Matt Sergeant
On Mon, 4 Aug 2008 15:27:46 +0200, Diego d'Ambra wrote: The latest version of qpsmtpd-prefork (revision 935) seems to miss some important patches, e.g.: - shared memory LOCK (qpsmtpd freezes) - clean shutdown of parent and any children (unable to restart qpsmtpd) - cleanup of STDIN

Re: Correcting plugin syntax errors

2008-07-29 Thread Matt Sergeant
On Tue, 29 Jul 2008, Jose Luis Martinez wrote: BTW: any comment on how to elaborate a testing framework. Comments from the QP-gurus would be helpful. For some in house code here that's similar to Qpsmtpd we basically have a data driven test system. You specify in a config file what plugins

Re: [qpsmtpd] Decisions, Decisions

2008-07-08 Thread Matt Sergeant
On Tue, 8 Jul 2008, David Nicol wrote: On Tue, Jul 8, 2008 at 3:14 PM, [EMAIL PROTECTED] wrote: Basically you just need an understanding of how async programming works - from there everything starts to become obvious. never block for IO. Work out the state so that you can drop everything

Re: [qpsmtpd] Decisions, Decisions

2008-07-08 Thread Matt Sergeant
On Tue, 8 Jul 2008, David Nicol wrote: On Tue, Jul 8, 2008 at 4:33 PM, Matt Sergeant [EMAIL PROTECTED] wrote: But do note what I've said here previously: async is for high CONCURRENCY not necessarily performance. Up to a certain level of concurrency prefork is faster. with SMTP one cares

Re: [qpsmtpd] Decisions, Decisions

2008-07-07 Thread Matt Sergeant
On Mon, 7 Jul 2008 22:30:50 + (UTC), [EMAIL PROTECTED] wrote: Looking at the wiki deployment page, it looks like I could be considering the apache or prefork versions. I'm leaning towards prefork - the page says it's faster, but there's no comparison between it and the apache version. Are

Re: questions re: async, perperl, new release

2008-06-27 Thread Matt Sergeant
On Fri, 27 Jun 2008 04:17:55 -0700, Ask Bjørn Hansen wrote: Has anyone run qpsmtpd under PersistentPerl (a.k.a SpeedyCGI). There is a mention of pperl, but I've never used it. qpsmtpd-forkserver is our closest equivalent. I don't know if anyone's been running it under pperl or

Interesting article on async-ness

2008-06-27 Thread Matt Sergeant
http://oubiwann.blogspot.com/2008/06/so-you-want-your-code-to-be-asynchonous.html It's about Python, but the same stuff generally applies to working with qpsmtpd-async.

Re: ax2: base-class in XSP documents fails

2008-06-25 Thread Matt Sergeant
On Wed, 25 Jun 2008, Martijn wrote: On Tue, Jun 17, 2008 at 4:39 PM, Matt Sergeant wrote: Fixed in SVN (I think). I still get an error when I use My::BaseClass::SubClass as a base-class, which inherits the handler method from My::BaseClass. Which makes sense: in _load_module() only My

Re: ax2: base-class in XSP documents fails

2008-06-25 Thread Matt Sergeant
On Wed, 25 Jun 2008, Martijn wrote: On Wed, Jun 25, 2008 at 2:18 PM, Martijn wrote: That's exactly what I did. Thanks. That's exactly what I did *wrong* I meant to say. :-) Speaking of XSP, would it be worth adding a line no warnings 'redefine'; to the XSP-blurb that is being evaluated?

Re: [sqlite] configure syntax error on HP

2008-06-22 Thread Matt Sergeant
On Sat, 21 Jun 2008 11:50:31 +0700, Dan wrote: > >> On Thu, 19 Jun 2008 12:05:56 -0400, D. Richard Hipp wrote: >>> >>> On Jun 19, 2008, at 11:49 AM, Matt Sergeant wrote: >>>> >>>> Note that there are some C++ style comments crept back into the

Re: [sqlite] configure syntax error on HP

2008-06-22 Thread Matt Sergeant
On Sat, 21 Jun 2008 11:50:31 +0700, Dan wrote: On Thu, 19 Jun 2008 12:05:56 -0400, D. Richard Hipp wrote: On Jun 19, 2008, at 11:49 AM, Matt Sergeant wrote: Note that there are some C++ style comments crept back into the code (I noticed in the amalgamation, so I can't give you a direct

Re: [sqlite] configure syntax error on HP

2008-06-20 Thread Matt Sergeant
On Thu, 19 Jun 2008 12:05:56 -0400, D. Richard Hipp wrote: > > On Jun 19, 2008, at 11:49 AM, Matt Sergeant wrote: >> >> Note that there are some C++ style comments crept back into the code >> (I >> noticed in the amalgamation, so I can't give you a direct po

Re: [sqlite] configure syntax error on HP

2008-06-20 Thread Matt Sergeant
On Thu, 19 Jun 2008 12:05:56 -0400, D. Richard Hipp wrote: On Jun 19, 2008, at 11:49 AM, Matt Sergeant wrote: Note that there are some C++ style comments crept back into the code (I noticed in the amalgamation, so I can't give you a direct pointer to them). This causes compile failures

Re: [sqlite] configure syntax error on HP

2008-06-19 Thread Matt Sergeant
On Wed, 18 Jun 2008 19:58:02 -0400, D. Richard Hipp wrote: > > On Jun 18, 2008, at 7:12 PM, Andrea Connell wrote: > >> I want to use the C API with a C++ class but when I try compiling... >> >> $ aCC -AA +W829 main.cpp sqlite3.c >> main.cpp: >> sqlite3.c: >> Error 482: "sqlite3.c", line 532 #

Re: [sqlite] configure syntax error on HP

2008-06-19 Thread Matt Sergeant
On Wed, 18 Jun 2008 19:58:02 -0400, D. Richard Hipp wrote: On Jun 18, 2008, at 7:12 PM, Andrea Connell wrote: I want to use the C API with a C++ class but when I try compiling... $ aCC -AA +W829 main.cpp sqlite3.c main.cpp: sqlite3.c: Error 482: sqlite3.c, line 532 # Array of unknown

Re: AxKit2 dvelopment

2008-06-17 Thread Matt Sergeant
On Tue, 17 Jun 2008 13:06:09 +0100, Martijn wrote: Hello all. Hi, Since this list has gone rather quiet, I was wondering what the development status of AxKit2 is. While working with AxKit2, I found a few (minor) bugs in the code (e.g. sending multiple cookies didn't work). I thought it

Re: ax2: base-class in XSP documents fails

2008-06-17 Thread Matt Sergeant
On Fri, 6 Jun 2008 15:48:59 +0100, Martijn wrote: Hello again. Not sure if this is a bug, but I can't get AxKit2 to use base classes in XSP documents, via the base-class attribute of the xsp:page root element. Whenever I try to add a perl module that contains only an empty handler

Re: AxKit to AxKit 2? / axkit 1 axkit uris issues.

2008-06-17 Thread Matt Sergeant
On Tue, 17 Jun 2008 14:12:55 -0700, fess wrote: I have a site that has been running AxKit 1.6.2 for some years now. I have 2 questions: 1. Is there any documentation about the difference between the Axkit 1 and AxKit2? Any how to upgrade, why to upgrade documents? There isn't. I

Re: require_resolvable_fromhost bug

2008-06-12 Thread Matt Sergeant
Heh, I was going through this code a week or so ago and thought this looked odd... I'll apply the patch. On Thu, 12 Jun 2008 14:53:22 +0200, Jose Luis Martinez wrote: Hi, We're using the require_resolvable_fromhost plugin, and have seen that there is a bug in it. MAIL FROM commands with

Re: rcptto_vrfy

2008-06-10 Thread Matt Sergeant
On Tue, 10 Jun 2008 13:59:43 +0200, Jan Völkers wrote: Hello, has anyone written a rcptto_vrfy plugin? I set up a new mx in a hurry and now i am searching for a nice check_rcpt plugin. Actually i am using only the smtp_forward plugin which has two disadvantages: - it checks _after_

Re: [svn:qpsmtpd] r923 - in trunk: . lib/Qpsmtpd

2008-06-03 Thread Matt Sergeant
On 3-Jun-08, at 4:04 AM, Radu Greab wrote: I think we need more consensus on this patch. Due to the objections expressed so far I reverted the change. Should I mention the overloaded meaning of remote_host in the pod? No, because it's different with different backends. Currently only

Re: Clarification of hooks with ignored return values

2008-06-03 Thread Matt Sergeant
On 3-Jun-08, at 4:36 AM, Radu Greab wrote: The documentation says that for the following hooks the return values are ignored or discarded: post-fork post-connection disconnect deny ok I had however to change my custom plugins to return DECLINE from post-fork hooks, otherwise, with no

Re: [svn:qpsmtpd] r923 - in trunk: . lib/Qpsmtpd

2008-06-02 Thread Matt Sergeant
On 2-Jun-08, at 6:06 PM, Radu Greab wrote: Matt Sergeant wrote: I think that remote_host should not be the error code in case of errors. That contradicts the description of the method and may force other people using remote_host to do their own checks. Well on regular qmail it's set

Re: Pulling my hair out...

2008-05-28 Thread Matt Sergeant
On Wed, 28 May 2008, Chris Lewis wrote: I'm trying to implement a subroutine shared between plugins. The library routine looks something like: sub createpattern {} It needs to be exported from your library, or called with a full package name, how are you doing that? The most common way

Re: Connection notes and TLS

2008-05-26 Thread Matt Sergeant
On 26-May-08, at 5:24 PM, John Peacock wrote: Those are *connection* attributes, not *transaction* attributes. If a plugin is storing them in the transaction object, that is a mistake. We can and probably should maintain the connection notes from before the TLS reset, but I maintain that

Re: Connection notes and TLS

2008-05-23 Thread Matt Sergeant
On 22-May-08, at 10:45 PM, John Peacock wrote: STARTTLS is not required to happen immediately after EHLO (not HELO, which doesn't support ESMTP extensions). And yes, you must completely discard every portion of the SMTP state that has occurred up to that point (just like with RSET). The

Re: Creating global DB connection

2008-05-20 Thread Matt Sergeant
On 20-May-08, at 12:33 PM, Nighthawk wrote: Hi, I am trying to write plugin that does DB lookup and want to use the same db connection in more than 1 plugins. Somwhere I read that we can save DB connection in qp-config(). But it is not working in my case. Store it in

Re: Whitelist convention

2008-05-16 Thread Matt Sergeant
Do it.

Re: Next steps to release?

2008-05-12 Thread Matt Sergeant
On Mon, 12 May 2008, Hanno Hecker wrote: On Wed, 7 May 2008 19:58:46 + (UTC) Matt Sergeant [EMAIL PROTECTED] wrote: So the next release I consider to be a fairly major step - we've got async/smtp-forward and async tls working. That's most of the showstoppers against using async

Re: [svn:qpsmtpd] r903 - trunk/lib/Qpsmtpd

2008-05-12 Thread Matt Sergeant
Note that this is probably worth doing for the other backends, though it's only for performance really. On Mon, 12 May 2008, [EMAIL PROTECTED] wrote: Author: msergeant Date: Mon May 12 10:19:31 2008 New Revision: 903 Modified: trunk/lib/Qpsmtpd/PollServer.pm Log: Make sure non-responding

Next steps to release?

2008-05-07 Thread Matt Sergeant
So the next release I consider to be a fairly major step - we've got async/smtp-forward and async tls working. That's most of the showstoppers against using async in production. So what's missing and what would you like to see before the next release? (obviously not limited to -async stuff)

Re: async and tls take two

2008-05-05 Thread Matt Sergeant
On 5-May-08, at 10:45 AM, Steve Kemp wrote: Seeing changes like this makes me wonder if we should consider running all source through perltidy at some point. Perhaps as part of a make update, or make commit target. Yes, it has been discussed before. We just need the tuits to make it

Re: async and tls take two

2008-05-04 Thread Matt Sergeant
On 4-May-08, at 7:28 PM, Matt Sergeant wrote: Yeah this makes a lot more sense... Though I wonder if we shouldn't modify Danga::Client to just have some sort of reader entry, so that anything else (not just TLS) can hook into the event_read stream. I'll have a poke. Can you check

Re: high CPU on lost processes with forkserver

2008-04-25 Thread Matt Sergeant
On 25-Apr-08, at 1:15 AM, Peter J. Holzer wrote: A better option might be to have the parent process watch for long running children and terminate them. Yup, but how long is long? If the client is trying to send a 600 MB email that will take some time ... Well we could set a max session

Re: high CPU on lost processes with forkserver

2008-04-24 Thread Matt Sergeant
On 24-Apr-08, at 5:52 PM, Brian Szymanski wrote: Ask Bjørn Hansen wrote: On Apr 24, 2008, at 11:02 AM, Charlie Brady wrote: Ask said Yeah, this is a pretty bad bug in March 2007, but I haven't seen anyone looking to fix it. We must be in pretty good shape when billions (or whatever) of

Re: qpsmtpd-async: premature end of data problem

2008-04-14 Thread Matt Sergeant
On Sat, 12 Apr 2008, Radu Greab wrote: I found a case where qpsmtpd-async detects the end of data marker incorrectly: the previous packet did not end with CRLF and the current packet starts with .CRLF. The code assumes that the previous packet ended with CRLF. Attached are a test script and a

Re: qpsmtpd-async: premature end of data problem

2008-04-14 Thread Matt Sergeant
On Mon, 14 Apr 2008, Matt Sergeant wrote: On Sat, 12 Apr 2008, Radu Greab wrote: I found a case where qpsmtpd-async detects the end of data marker incorrectly: the previous packet did not end with CRLF and the current packet starts with .CRLF. The code assumes that the previous packet ended

Re: qpsmtpd-async: premature end of data problem

2008-04-14 Thread Matt Sergeant
On Sat, 12 Apr 2008, Radu Greab wrote: I found a case where qpsmtpd-async detects the end of data marker incorrectly: the previous packet did not end with CRLF and the current packet starts with .CRLF. The code assumes that the previous packet ended with CRLF. Attached are a test script and a

Re: qpsmtpd-async: premature end of data problem

2008-04-14 Thread Matt Sergeant
On Mon, 14 Apr 2008, Guy Hulbert wrote: On Mon, 2008-14-04 at 12:54 +, Matt Sergeant wrote: Attached are a test script and a suggested patch. I committed the patch 95% the same as yours. I can't help feeling there's a better way to do it - I hate applying two regexps every time a DATA

Re: qpsmtpd-async: premature end of data problem

2008-04-13 Thread Matt Sergeant
Ask Bjørn Hansen wrote: On Apr 12, 2008, at 6:33, Radu Greab wrote: I found a case where qpsmtpd-async detects the end of data marker incorrectly: the previous packet did not end with CRLF and the current packet starts with .CRLF. The code assumes that the previous packet ended with CRLF.

Re: disabling plugins

2008-04-09 Thread Matt Sergeant
Hanno Hecker wrote: Hi, this patch sets a way to disable any loaded plugin(s) for the current client. I'm not going to commit this before Ask released 0.43. Matt (and others with high traffic) should be ok with the one more line next unless $code-{run}; per hook. The plugins_loaded() should

Re: disabling plugins

2008-04-09 Thread Matt Sergeant
Hanno Hecker wrote: seems like a rather heavy hammer, which can be influenced in a very bizarre cross connection manner. Except for using this before the 'connect' hook, I don't see where. But ...(thinking loud) maybe use the plugins_loaded() and ## Qpsmtpd.pm, run_continuation():

Re: [PATCH] SMTP-AUTH broken

2008-04-08 Thread Matt Sergeant
On Mon, 31 Mar 2008, John Peacock wrote: but I don't know what Matt was trying to fix. As Ask surmised, the whole point was to have the object hooks, not a package hooks, be used, since the hooks registered are attributes of the object, not the class. Attached is a diff that fixes both tls

Re: [PATCH] SMTP-AUTH broken

2008-03-31 Thread Matt Sergeant
On 31-Mar-08, at 7:34 AM, John Peacock wrote: Attached is a diff that fixes both tls and auth, by backing out 814 and tweaking a couple of other lines. I won't commit it until Matt pipes up why he tried to make hooks a package global... Because otherwise hooks are re-loaded on every

Re: [PATCH] SMTP-AUTH broken

2008-03-31 Thread Matt Sergeant
On 31-Mar-08, at 10:12 AM, John Peacock wrote: Matt Sergeant wrote: Because otherwise hooks are re-loaded on every connection (register () needs re-run on every mail) which is a huge performance detriment. Then we need to rethink that architecture; would it be acceptable to return

Re: [PATCH] SMTP-AUTH broken

2008-03-31 Thread Matt Sergeant
On 31-Mar-08, at 11:04 AM, John Peacock wrote: It was because AUTH might have two roundtrips with the client (or not) depending on the AUTH method used. The first AUTH command is clearly defined (since it starts with AUTH ;-), but the client response to the server challenge has no obvious

Re: Release

2008-03-18 Thread Matt Sergeant
On 14-Mar-08, at 9:48 AM, Matt Sergeant wrote: Ask and Matt What's the current release status of qpsmtpd? The SVN is tagged 0.43 but the latest on the website seems to be 0.40. Time for a release and announce? Sounds like a plan. In case anyone (well, Ask) was thinking about cutting

Re: Release

2008-03-18 Thread Matt Sergeant
On 18-Mar-08, at 9:53 AM, Matt Sergeant wrote: On 14-Mar-08, at 9:48 AM, Matt Sergeant wrote: Ask and Matt What's the current release status of qpsmtpd? The SVN is tagged 0.43 but the latest on the website seems to be 0.40. Time for a release and announce? Sounds like a plan

Re: Release

2008-03-14 Thread Matt Sergeant
On 14-Mar-08, at 4:54 AM, James Turnbull wrote: Ask and Matt What's the current release status of qpsmtpd? The SVN is tagged 0.43 but the latest on the website seems to be 0.40. Time for a release and announce? Sounds like a plan.

Re: [PATCH] qpsmtpd-async: set supplementary UNIX groups

2008-03-13 Thread Matt Sergeant
Applied, thanks On 10-Mar-08, at 5:09 PM, Radu Greab wrote: I found that qpsmtpd-async can not deliver mail with queue/postfix- queue due to lack of permissions, the qpsmtpd-async process did not belong to the postdrop group. Here is a patch to update qpsmtpd-async to set the

Re: [PATCH] plugins/async/require_resolvable_fromhost improvements

2008-03-13 Thread Matt Sergeant
Applied, thanks. On 10-Mar-08, at 6:29 PM, Radu Greab wrote: I found that the async/require_resolvable_fromhost plugin has two problems: - it sometimes denies mails from resolvable fromhosts - it may call $qp-run_continuation multiple times, making other plugins run too early I believe

Re: [PATCH] make async/require_resolvable_fromhost accept mail from null sender

2008-03-13 Thread Matt Sergeant
Applied, thanks. On 11-Mar-08, at 2:02 PM, Radu Greab wrote: If async/require_resolvable_fromhost is used, then mail from the null sender is rejected. Please check the attached patch: the second part of the plugin will DECLINE if the first part declined too because of the null sender.

Re: [PATCH] restore the timers after Danga::Socket-Reset()

2008-03-13 Thread Matt Sergeant
Good catch. Applied. On 13-Mar-08, at 4:46 PM, Radu Greab wrote: Danga::Socket-Reset() deletes all timers. Idle SMTP connections do not time out anymore. The attached patch restores the timers. Thank you, Radu Greab qpsmtpd-0.43-async-restore_timers.patch

Re: [PATCH] to add qpsmtpd-prefork to the list of EXE_FILES in the Makefile

2008-03-12 Thread Matt Sergeant
On Wed, 12 Mar 2008, Ask Bjørn Hansen wrote: Perhaps qpsmtpd-async should be added too? Matt, what do you think? Definitely.

Re: qpsmtpd performance

2008-03-12 Thread Matt Sergeant
On Wed, 12 Mar 2008, Bryan Scott wrote: Forkserver spawns and forks a bunch of qpsmtpd processes ahead of time instead of the shell having to spawn a new perl process each time a connection comes in. Not quite - that's prefork :-) Forkserver forks on every connection, but has the perl code

Re: [PATCH] qpsmtpd-async: close unneeded sockets in children

2008-03-11 Thread Matt Sergeant
On 11-Mar-08, at 2:14 PM, Radu Greab wrote: Am I right to suppose that a better option would be to have the config port open and watched only in the parent and when someone connects to it and issues commands the parent will relay the commands to _all_ children, will wait for all results and

Re: qpsmtpd performance

2008-03-11 Thread Matt Sergeant
On 11-Mar-08, at 1:53 PM, Jens Weibler wrote: I can't imagine that you've so much spam connections at the same time. Concurrency on our (qpsmtpd-async) spamtrap goes up to 3000 several times a day. I've seen it hit 10k. Be thankful if you're NOT getting that sort of concurrency :-) Matt.

Re: qpsmtpd performance

2008-03-11 Thread Matt Sergeant
On 11-Mar-08, at 3:10 PM, Juerd Waalboer wrote: Matt Sergeant skribis 2008-03-11 15:11 (-0400): On 11-Mar-08, at 1:53 PM, Jens Weibler wrote: I can't imagine that you've so much spam connections at the same time. Concurrency on our (qpsmtpd-async) spamtrap goes up to 3000 several times

Re: qpsmtpd performance

2008-03-11 Thread Matt Sergeant
On 11-Mar-08, at 5:00 PM, Juerd Waalboer wrote: Matt Sergeant skribis 2008-03-11 16:01 (-0400): Compaq (HP) G3. Dual Intel(R) Xeon(TM) CPU 3.06GHz (so 4 CPUs under hyperthreading). Load is spread across 4 mail servers (...) Impressive; must have been nice to design the setup.. Thanks

Re: new plugin: check_system_load

2008-03-08 Thread Matt Sergeant
On 8-Mar-08, at 3:21 PM, Brian Szymanski wrote: eval { require Sys::CpuLoad }; my $_has_sys_cpuload = !$@; You might find this meme useful: use constant HAVE_CPULOAD = eval { require Sys::CpuLoad };

Re: new plugin: check_system_load

2008-03-07 Thread Matt Sergeant
Note - you'd be better off using Linux::SysInfo (on Linux, obviously) to save the backtick call. On 7-Mar-08, at 4:39 PM, Steve Kemp wrote: This seems like rather an obvious idea: Reject new connections if the system load is too high. But surprisingly there seem to be no plugins which

Re: async/check_earlytalker: AddTimer

2008-02-26 Thread Matt Sergeant
On 25-Feb-08, at 4:35 PM, Jared Johnson wrote: Using async/check_earlytalker from qp 0.40, I get the following error: FATAL PLUGIN ERROR: Can't locate object method AddTimer via package Qpsmtpd::TcpServer at /usr/share/qpsmtpd/plugins/async/ check_earlytalker line 94. grep can't find

Re: config wierdness

2008-02-19 Thread Matt Sergeant
On Sat, 16 Feb 2008, Matt Sergeant wrote: Isn't this fixed by r845? Answer from Chris here at MAAWG: Yes it does :-) Matt.

Re: config wierdness

2008-02-16 Thread Matt Sergeant
On Thu, 14 Feb 2008, Chris Lewis wrote: Like Solaris usually does (grr), hostname returns the host node name (no dots), not the FQDN. config/me contains the FQDN, but $self-qp-config(me) _still_ returns the node name. I see the sub config code uses `hostname` as the default, but shouldn't

Re: [qpsmtpd] how to accept msg and drop or redirect?

2008-02-13 Thread Matt Sergeant
On 13-Feb-08, at 3:31 PM, [EMAIL PROTECTED] wrote: I'm using forkserver 0.32. I've have a need to either drop or redirect certain messages after accepting them. In other words, I do not want to reject the connection, but I also do not want to make the delivery to the provided recipient

Re: [qpsmtpd] how to accept msg and drop or redirect?

2008-02-13 Thread Matt Sergeant
On 13-Feb-08, at 4:34 PM, [EMAIL PROTECTED] wrote: On Wed, 13 Feb 2008, Matt Sergeant wrote: Just create a queue plugin which based on whatever criteria you have for dropping returns OK. Run this plugin *before* your real queue plugin. Thanks for the reply. I notice that the 'relay

Re: [qpsmtpd] how to accept msg and drop or redirect?

2008-02-13 Thread Matt Sergeant
On 13-Feb-08, at 5:19 PM, [EMAIL PROTECTED] wrote: That's a different hook. I'm not sure I follow you. Do you mean that the relay plugin hooks rcpt, and the queue plugin hooks queue, so the OK returned in the relay plugin doesn't have the same effect as the OK returned in the queue

Re: AxKit and Apache version

2008-02-03 Thread Matt Sergeant
On 3-Feb-08, at 4:47 AM, Vaclav Barta wrote: On Sunday 03 February 2008 01:11:57 Matt Sergeant wrote: On 2-Feb-08, at 1:58 PM, Jean-Michel Caricand wrote: I'm new with AxKit. I have a little question. Can I use AxKit 1.7.0 with mod_perl 2 ? Nope. Your best bet is to proxy through Apache2

Re: AxKit and Apache version

2008-02-02 Thread Matt Sergeant
On 2-Feb-08, at 1:58 PM, Jean-Michel Caricand wrote: I'm new with AxKit. I have a little question. Can I use AxKit 1.7.0 with mod_perl 2 ? Nope. Your best bet is to proxy through Apache2 onto an Apache 1.3.x backend with AxKit on. Matt.

Re: qpsmtpd-async and dnsbl

2008-01-31 Thread Matt Sergeant
On 30-Jan-08, at 5:14 PM, mark warren bracher wrote: Matt Sergeant wrote: On 29-Jan-08, at 12:55 PM, [EMAIL PROTECTED] wrote: So I'll need more info about your configuration. ok, found something. this only happens, when I use logging/ adaptive. If I take that out, the dnsbl starts to work

Re: qpsmtpd-async and dnsbl

2008-01-30 Thread Matt Sergeant
On 29-Jan-08, at 12:55 PM, [EMAIL PROTECTED] wrote: So I'll need more info about your configuration. ok, found something. this only happens, when I use logging/adaptive. If I take that out, the dnsbl starts to work. Can you send the output from enabling verbose mode (two -v flags) with

Re: qpsmtpd-async and dnsbl

2008-01-29 Thread Matt Sergeant
On 29-Jan-08, at 2:53 AM, [EMAIL PROTECTED] wrote: What steps will reproduce the problem? qpsmtpd-async(rev 838) with dnsbl plugin enabled The Problem is, qpsmtpd will never print out the initial 220, so that the smtp dialog can start. I don't see that: $ cat config/plugins async/dnsbl

Re: qpsmtpd-async and dnsbl

2008-01-29 Thread Matt Sergeant
On 29-Jan-08, at 11:42 AM, mark warren bracher wrote: same config here, except I'm using qmail-queue. OK - let me align versions. What version of qpsmtpd and what version of ParaDNS?

<    1   2   3   4   5   6   7   8   9   10   >