Re: No such pseudo-hash field conn ...

2005-07-18 Thread Matt Sergeant
Now fixed in SVN. On 17 Jul 2005, at 15:59, Matthew Harrell wrote: Admittedly I probably shouldn't be using the latest and greated in the subversion repository but I figured I would report this problem since I hadn't seen a fix in the last week. When I try to connect to my qpsmtpd process

Re: tls returns undef at mail rcpt and data hooks if unused

2005-07-18 Thread Matt Sergeant
Fixed in SVN. On 17 Jul 2005, at 16:44, Bob Dodds wrote: sub bad_ssl_hook() needs to end with the default return DECLINED; -Bob

Re: rcpthosts in config.sample

2005-07-18 Thread Matt Sergeant
On 18 Jul 2005, at 13:47, John Peacock wrote: Yeah, if we intend to be the Inbound SMTP Interface to the World (ISIttW doesn't really work as a project name, though ;-), we should probably add one. However, it should include a paragraph on what to put there, as well as the single line:

rcpthosts in config.sample

2005-07-18 Thread Matt Sergeant
Since we absolutely rely on rcpthosts, should we add one to config.sample? I'm guessing we don't have one by default because qmail already has a rcpthosts file. But we seem to have moved quite a bit away from qmail now. Matt.

Re: [PATCH] Missing tie() call in NDBM_File SYNOPSIS

2005-07-15 Thread Matt Sergeant
On 15 Jul 2005, at 03:20, H.Merijn Brand wrote: On Thu, 14 Jul 2005 16:53:03 -0400, Matt Sergeant [EMAIL PROTECTED] wrote: Simple patch for an obvious omission (also fixes the lack of whitespace on the line before untie()): In this patch I see no whitespace changes at all, nor do I see

Re: [PATCH] Missing tie() call in NDBM_File SYNOPSIS

2005-07-15 Thread Matt Sergeant
On 15 Jul 2005, at 08:03, H.Merijn Brand wrote: If you think this is worthwhile, please send a new attached patch. Note however the subsequent patches might act more or less like what I experienced and the trailing spaces get lost again. Your call Nah, I was just letting you know why I did

Re: standard asynchronous deferred-to-result syntax RFP

2005-07-15 Thread Matt Sergeant
On 14 Jul 2005, at 13:16, David Nicol wrote: I have uploaded asynchronous::universal::ready and asynchronous::universal::set_callback to CPAN. They are both entirely trivial packages with a mess of documentation. The idea behind them is to support asynchonous frameworks in which the immediate

[PATCH] Apache::Test can never use mp1 with mp2 installed

2005-07-15 Thread Matt Sergeant
There's a bug in Apache::Test that prevents it from working properly when you're trying to test something with mp1 when you've got mp2 installed. What happens is that Apache::TestConfig does this: use constant IS_MOD_PERL_2 = eval { require mod_perl2 } || 0; Which of course is

Re: standard asynchronous deferred-to-result syntax RFP

2005-07-15 Thread Matt Sergeant
On 14 Jul 2005, at 13:16, David Nicol wrote: I have uploaded asynchronous::universal::ready and asynchronous::universal::set_callback to CPAN. They are both entirely trivial packages with a mess of documentation. The idea behind them is to support asynchonous frameworks in which the immediate

[PATCH] Apache::Test can never use mp1 with mp2 installed

2005-07-15 Thread Matt Sergeant
There's a bug in Apache::Test that prevents it from working properly when you're trying to test something with mp1 when you've got mp2 installed. What happens is that Apache::TestConfig does this: use constant IS_MOD_PERL_2 = eval { require mod_perl2 } || 0; Which of course is

[PATCH] Missing tie() call in NDBM_File SYNOPSIS

2005-07-14 Thread Matt Sergeant
Simple patch for an obvious omission (also fixes the lack of whitespace on the line before untie()): --- ext/NDBM_File/NDBM_File.pm.orig Thu Jul 14 16:49:22 2005 +++ ext/NDBM_File/NDBM_File.pm Thu Jul 14 16:50:36 2005 @@ -23,12 +23,15 @@ use Fcntl; # For O_RDWR, O_CREAT, etc. use

Re: About smtpgreeting

2005-07-14 Thread Matt Sergeant
On 14 Jul 2005, at 14:51, Guillaume Filion wrote: I'm wondering if one of the smtpgreeting patches is going to be included in svn. I don't really mind which one is used (1) but I would find that feature useful. Anyone has reserves about this? Hmm, #1 looks really interesting... I missed it

Re: High_perf has landed

2005-07-12 Thread Matt Sergeant
On 12 Jul 2005, at 13:05, Bryan Scott wrote: Matt Sergeant wrote: Trunk now contains high_perf. - Please try it, especially qpsmtpd-forkserver users since the forkserver code changed too, and I'd like to know if it still all works OK. I have done some rudimentary testing, but it could

Re: High_perf has landed

2005-07-12 Thread Matt Sergeant
On 12 Jul 2005, at 13:55, Bryan Scott wrote: With so much going on with qpsmtpd, it's hard to know what to use! :) (Exciting this is.) If you want high-perf then use the qpsmtpd script, not qpsmtpd-forkserver. I guess I broke qpsmtpd-forkserver, so I'm going to go back to a more stable way

Sticky problem with waitpid/REAPER

2005-07-11 Thread Matt Sergeant
Just came across a really thorny problem with our implementation of qmail-queue. Basic summary is that we have a potential race condition between calls to waitpid() in the parent process and in qmail-queue. If the parent process' waitpid(-1) call enters before the child processes

Re: Sticky problem with waitpid/REAPER

2005-07-11 Thread Matt Sergeant
On 11 Jul 2005, at 11:43, Robert Spier wrote: Anyone have any thoughts on how to solve this? Can you tell waitpid() to only operate on child processes and not grandchild processes? No, but you can tell it to only wait for a specific pid. But that won't help in this case. Ah actually I

Re: Design question

2005-07-10 Thread Matt Sergeant
On 10 Jul 2005, at 15:19, John Peacock wrote: So there seem to be a couple of ways to approach this: 1) Add either a $transaction-{_connection} or $transaction-{_qp} element, so that the transaction can crawl back up to it's containing objects to log using the cached logging plugin. 2)

Re: [Patch] STARTTLS support (forkserver only)

2005-07-08 Thread Matt Sergeant
On 8 Jul 2005, at 11:42, Charlie Brady wrote: On Fri, 8 Jul 2005, Matt Sergeant wrote: Can you try the attached plugin, along with *just* your change to reset the connection. Will do. Note there are some bugs in the plugin I sent you. I'll work on them until I can get things working

Re: Testing STARTTLS (Re: [Patch] STARTTLS support (forkserver only))

2005-07-08 Thread Matt Sergeant
On 8 Jul 2005, at 13:36, Charlie Brady wrote: For those interested in dabbling with STARTTLS, it's very useful to have a convenient scriptable testing tool. I've been using swaks (http://jetmore.org/john/code/#swaks) which has been extremely useful. I'll check it out.

Re: [Patch] STARTTLS support (forkserver only)

2005-07-08 Thread Matt Sergeant
On 8 Jul 2005, at 12:49, Charlie Brady wrote: Biggest problem seems to be that the password is requested every time. The STARTTLS plugin you sent doesn't have anything to do with passwords. So you are talking about some interaction with AUTH, which I haven't looked at. No, this is to do

Re: [Patch] STARTTLS support (forkserver only)

2005-07-08 Thread Matt Sergeant
On 8 Jul 2005, at 13:10, John Peacock wrote: Matt Sergeant wrote: I think it'd be easy to do using isa_plugin - just have hook that is an overridden method in the subclass. Then you just have the base class hook into unrecognized_command as I did with STARTTLS. No, I mean that I could have

Re: [perl #36486] Bug Test

2005-07-08 Thread Matt Sergeant
On 8 Jul 2005, at 22:20, Keith Ivey wrote: The date line was missing but my mail server added one, so I didn't notice. Maybe that's something qpsmtpd should do too :-)

Re: PATCH: Direct bugs to the list, not to ask

2005-07-07 Thread Matt Sergeant
On 7 Jul 2005, at 01:09, Gordon Rowell wrote: Or should we have a bug reporting address/tracker? We should. I think there's RT setup at perl.org for this...

Re: [svn:qpsmtpd] rev 479 - in trunk: . lib/Qpsmtpd plugins plugins/auth plugins/ident plugins/logging plugins/queue plugins/virus

2005-07-07 Thread Matt Sergeant
On 7 Jul 2005, at 00:17, [EMAIL PROTECTED] wrote: +Instead of modifying @ISA directly in your plugin, use the +C plugin_isa method from the init subroutine. + + # rcpt_ok_child + sub init { +my ($self, $qp) = @_; +$self-isa_plugin('rcpt_ok'); + } Which is it? isa_plugin or

Re: RC scripts

2005-07-07 Thread Matt Sergeant
On 7 Jul 2005, at 18:03, Gordon Rowell wrote: We (SMEServer) have single RC script called daemontools and we symlink all of our daemontools/runit processes to that. All daemon specific startup is in the run scripts. Anyway, attached. Hmm, I'm thinking now that forkserver creates PID files,

Re: [sqlite] Does SQLite have a fulltext search like MySQL?

2005-07-06 Thread Matt Sergeant
On 5 Jul 2005, at 17:48, Michael Grice wrote: If not, are there plans to add this? What language are you planning to use? Perl has a bunch of full text search modules that implement FTS on top of any DB. Matt. __ This

Re: PATCH: let forkserver listen on multiple interfaces

2005-07-06 Thread Matt Sergeant
On 6 Jul 2005, at 15:40, Charlie Brady wrote: On Tue, 5 Jul 2005, Devin Carraway wrote: Forkserver can accept a --listen-address switch to listen on a particular address/interface instead of 0.0.0.0, but only one. This expands the handling of that switch so as to listen on any number of

unrecognized_command does disconnect()

2005-07-06 Thread Matt Sergeant
I've just gone through fixing a few documentation snafu's, and noticed in SMTP.pm that if you return DENY from unrecognized_command it does a -disconnect(). Do we think that's intentional? It seems to me if you want a disconnect you'd ask for it with DENY_DISCONNECT. Thoughts?

Re: PATCH: let forkserver listen on multiple interfaces

2005-07-06 Thread Matt Sergeant
On 6 Jul 2005, at 16:46, John Peacock wrote: Charlie Brady wrote: That's an added complication, and more code - which usually means that more things can go wrong. In what circumstances would someone want more than one interface, but not all interfaces? I was wondering about this myself and

Re: RFC: Use Class::Frame (was Re: Great news from YAPC)

2005-07-06 Thread Matt Sergeant
On 6 Jul 2005, at 16:51, Charlie Brady wrote: Thoughts? I'd rather just eval those subs into existence than need another CPAN module to do something so easy. Something like: BEGIN { my @fields = qw( remote_host remote_ip remote_info remote_port local_ip local_port

Re: PATCH: let forkserver listen on multiple interfaces

2005-07-06 Thread Matt Sergeant
On 6 Jul 2005, at 17:33, Ask Bjørn Hansen wrote: Should we plan on packaging up 0.31 in a week or two? Is the non-sig-handlers change not going into 0.30 then? Basically as soon as 0.30 is tagged and shipped I'm going to start merging high_perf in. Also I really really want you to get the

Re: unrecognized_command does disconnect()

2005-07-06 Thread Matt Sergeant
On 6 Jul 2005, at 17:53, Ask Bjørn Hansen wrote: DENY doesn't make sense at all in that context (the default is deny). Yes it does. The patch isn't right. You return DENY, $msg if you want a custom message.

Re: PATCH: config and plugin directories

2005-07-06 Thread Matt Sergeant
On 6 Jul 2005, at 20:47, Gavin Carr wrote: What about just using named arguments and honouring 'nolog = 1' or something? Yeah I guess we may have to bite the bullet on that soon. We talked about it a while ago. The only downside is that they're not used anywhere else in the source so it's a

Re: [sqlite] Does SQLite have a fulltext search like MySQL?

2005-07-06 Thread Matt Sergeant
On 5 Jul 2005, at 17:48, Michael Grice wrote: If not, are there plans to add this? What language are you planning to use? Perl has a bunch of full text search modules that implement FTS on top of any DB. Matt. __ This

Re: PATCH: let forkserver listen on multiple interfaces

2005-07-05 Thread Matt Sergeant
On 5 Jul 2005, at 06:08, Devin Carraway wrote: And since I was one line away -- is there any remaining good reason why the default port for forkserver is 2525? This has the look of development or testing leftovers. Yes, it makes you have to think before deploying. I like that the default

Re: Changing the Plugin APIs

2005-07-02 Thread Matt Sergeant
On 2 Jul 2005, at 03:14, Robert Spier wrote: Or the other way around - if the hook_* functions exist, have register_hook called magically behind the scenes when you lookup whether it can do the hooks. Good idea. This is cleaner. Patch below. Should we wait until after 0.30? I know it's

Great news from YAPC

2005-06-30 Thread Matt Sergeant
A birdy tells me that qpsmtpd will be the default smtp server in the next release of the e-smith SME server. Apparently that's 10K new users (not sure whether that was end-users or actual admins).

Re: PATCH: high_perf: pipelined commands force a race

2005-06-23 Thread Matt Sergeant
On 23 Jun 2005, at 04:09, Brian Grossman wrote: Is {can_read_mode} useful anymore? Hopefully not.

Re: PATCH: high_perf: respond methods want a msg arg

2005-06-23 Thread Matt Sergeant
On 23 Jun 2005, at 04:13, Brian Grossman wrote: After a continuation where the next called hook didn't provide a second arg to return, the corresponding respond method would get confused. Can you explain a bit more about this? Perl pretty much always autovivifies an undef, so I can't see

Anyone in Norway?

2005-06-23 Thread Matt Sergeant
Apparently there's an article on qpsmtpd in Norway's Linux Magasinet this month. Print only. Matt.

Re: PATCH: high_perf: respond methods want a msg arg

2005-06-23 Thread Matt Sergeant
On 23 Jun 2005, at 15:20, Brian Grossman wrote: The problem manifests a few lines below that: return $meth-($self, @r, @$args); If @r has only one element, then the first element of @$args will be used as $msg in *_respond(). Gotcha. I think it might be easier to just be explicit

Re: PATCH: high_perf: pipelined commands force a race

2005-06-23 Thread Matt Sergeant
On 23 Jun 2005, at 04:09, Brian Grossman wrote: If a client pipelined commands, process_read_buf would rush right through all of them (ignoring disable_read), forcing the later commands to (try to) execute even through we're waiting for a continuation. I'm not sure whether this patch is the

Re: [sqlite] Quick news on the Perl-bindings front

2005-06-21 Thread Matt Sergeant
On 21 Jun 2005, at 15:41, Darren Duncan wrote: At 1:29 PM -0400 6/21/05, Matt Sergeant wrote: 1.09 is now on CPAN. Note that there's a weird bug when trying to compile against the system sqlite on OS X Tiger due to some munging Apple have done to the header files. Someone is supplying me

Re: [sqlite] Quick news on the Perl-bindings front

2005-06-21 Thread Matt Sergeant
On 20 Jun 2005, at 06:57, Randy J. Ray wrote: I just sent a patch to the maintainer of the DBD::SQLite package, that lets it build against an installed version of the library. The current package carries a copy of the code with it, and builds it locally. With this patch, you can update your

Re: [sqlite] Quick news on the Perl-bindings front

2005-06-21 Thread Matt Sergeant
On 20 Jun 2005, at 06:57, Randy J. Ray wrote: I just sent a patch to the maintainer of the DBD::SQLite package, that lets it build against an installed version of the library. The current package carries a copy of the code with it, and builds it locally. With this patch, you can update your

Re: [sqlite] Quick news on the Perl-bindings front

2005-06-21 Thread Matt Sergeant
On 21 Jun 2005, at 15:41, Darren Duncan wrote: At 1:29 PM -0400 6/21/05, Matt Sergeant wrote: 1.09 is now on CPAN. Note that there's a weird bug when trying to compile against the system sqlite on OS X Tiger due to some munging Apple have done to the header files. Someone is supplying me

Re: Scoreboard plans?

2005-06-21 Thread Matt Sergeant
On 21 Jun 2005, at 11:43, Malcolm J Harwood wrote: On Monday 20 June 2005 05:36 pm, Matt Sergeant wrote: I see in the docs there's a stub to return $c-sbh - a scoreboard handle. Are there any plans to implement this any time soon? Which docs? Apache2::Connection - right at the end. OK

Re: Scoreboard plans?

2005-06-21 Thread Matt Sergeant
On 21 Jun 2005, at 14:56, Malcolm J Harwood wrote: I'm not sure if it makes sense to have it in Apache2::Connection, as the WorkerScore is process/thread based rather than connection based. (It does have connection specific information in it, but it also has more long lived data). I'm

Re: Loop detection

2005-06-20 Thread Matt Sergeant
On 20 Jun 2005, at 09:31, Keith Ivey wrote: I've attached a plugin that duplicates the loop-checking in qmail-smtpd. Added to svn. Thanks.

Re: qpsmtpd processes in an infinite loop

2005-06-20 Thread Matt Sergeant
On 20 Jun 2005, at 12:49, Joe Schaefer wrote: So we don't need to delete/remove the bucket at all? Sweet - that should be even faster :-) I don't think it's correct though- how will $bb-is_empty ever be false without something like $b-delete? Hmm, maybe it's time to start using Apache::Test

Re: qpsmtpd processes in an infinite loop

2005-06-20 Thread Matt Sergeant
On 20 Jun 2005, at 13:57, Justin Erenkrantz wrote: --On Monday, June 20, 2005 1:50 PM -0400 Matt Sergeant [EMAIL PROTECTED] wrote: OK, so looking at the latest mod_perl sources we can dump all this bucket stuff - the APR::Socket now has a recv() method so we can just read straight from

Re: qpsmtpd processes in an infinite loop

2005-06-20 Thread Matt Sergeant
On 20 Jun 2005, at 14:07, Peter Eisch wrote: Is there a way to share the raw socket with APR::Socket for just a short time? At the beginning of the tcp session it would be nice to sit on a select() for a bit and ensure the client doesn't try forcing down a session. They're not supposed to

Re: qpsmtpd processes in an infinite loop

2005-06-20 Thread Matt Sergeant
On 20 Jun 2005, at 15:16, Peter Eisch wrote: Thanks Matt. I'm guessing this is only in svn rather than cvs? Yes, CVS was dropped a while ago.

Scoreboard plans?

2005-06-20 Thread Matt Sergeant
I see in the docs there's a stub to return $c-sbh - a scoreboard handle. Are there any plans to implement this any time soon? Reason I ask is that apache.org has now switched it's mail to using Apache::Qpsmtpd - yes that's right - a mail server built on mod_perl 2.0 - you guys should be proud

Re: Scoreboard plans?

2005-06-20 Thread Matt Sergeant
On 20 Jun 2005, at 17:36, Matt Sergeant wrote: I see in the docs there's a stub to return $c-sbh - a scoreboard handle. Are there any plans to implement this any time soon? OK, so answering my own question - looks like Apache::Scoreboard already works on mod-perl2, modulo a bunch of patches

Re: qpsmtpd processes in an infinite loop

2005-06-18 Thread Matt Sergeant
On 18 Jun 2005, at 12:08, Peter Eisch wrote: Ok, my turn to learn, where I have: while (!$bb-is_empty) { my $b = $bb-first; $b-remove; $b-read(my $newdata); $data .= $newdata; return $data if index($data, \n) = 0; }

[high_perf] Continuation support

2005-06-18 Thread Matt Sergeant
I've checked in support for a rudimentary form of continuations. The basic idea is that previously you had to rely on the following sequence of events: ((( hook_connect : dnsbl - fire off dns query and turn off read mode on socket. callback : get dns response - turn on read mode on

Re: qpsmtpd processes in an infinite loop

2005-06-17 Thread Matt Sergeant
On 17 Jun 2005, at 00:35, Joe Schaefer wrote: [EMAIL PROTECTED] (Justin Erenkrantz) writes: On Thu, Jun 16, 2005 at 11:28:43AM -0400, Matt Sergeant wrote: On 15 Jun 2005, at 14:47, Justin Erenkrantz wrote: As I've mentioned before, we (apache.org) are seeing our qpsmtpd processes stuck

Re: AxKit: Archives

2005-06-16 Thread Matt Sergeant
On 15 Jun 2005, at 06:55, Anthony Gardner wrote: I think I read on a mail that the axkit archives have been moved. Is that so? Thing is, I can't check the archives at axkit.org to find out as I'm sure I saw it after Jan 2005. I can't figure out why the axkit.org archives aren't working. If

Re: [sqlite] preventing text to integer conversion of bind variables in perl

2005-06-15 Thread Matt Sergeant
On 15 Jun 2005, at 17:02, Jonathan H N Chin wrote: So perhaps the check no longer performs a useful function now that sqlite allows one to specify the data type of the column? Perhaps indeed. I think I'll remove it from the next release. Matt.

Re: [sqlite] preventing text to integer conversion of bind variables in perl

2005-06-15 Thread Matt Sergeant
On 15 Jun 2005, at 11:56, Jonathan H N Chin wrote: I would be interested to know what version of DBD::SQLite Puneet Kishor is using, since I believe I have tracked the issue to a test in the sqlite_st_execute() function in dbdimp.c : else if (looks_like_number(value)) { /* bind

Re: [sqlite] preventing text to integer conversion of bind variables in perl

2005-06-15 Thread Matt Sergeant
On 15 Jun 2005, at 11:56, Jonathan H N Chin wrote: I would be interested to know what version of DBD::SQLite Puneet Kishor is using, since I believe I have tracked the issue to a test in the sqlite_st_execute() function in dbdimp.c : else if (looks_like_number(value)) { /* bind

Re: [sqlite] preventing text to integer conversion of bind variables in perl

2005-06-15 Thread Matt Sergeant
On 15 Jun 2005, at 17:02, Jonathan H N Chin wrote: So perhaps the check no longer performs a useful function now that sqlite allows one to specify the data type of the column? Perhaps indeed. I think I'll remove it from the next release. Matt.

Re: NOT reject authenicated users?

2005-06-08 Thread Matt Sergeant
On 8 Jun 2005, at 07:04, Michael Holzt wrote: A more generic approach which should probably be done with the stock plugins is to add a parameter which allows one to turn off this plugin for relaying and/or authenticated users. Actually I was thinking of adding a new return code which would

Re: NOT reject authenicated users?

2005-06-08 Thread Matt Sergeant
On 8 Jun 2005, at 10:01, Michael Holzt wrote: Actually I was thinking of adding a new return code which would skip past everything to just queue the mail. Something like: I would have no use for that, because i want some plugins to run for all users, e.g. a plugin checking if a mail address

Should we make plugins strict

2005-05-25 Thread Matt Sergeant
We currently don't use strict for plugins. Anyone object to me making it so?

Re: Should we make plugins strict

2005-05-25 Thread Matt Sergeant
On 25 May 2005, at 11:28, John Peacock wrote: Matt Sergeant wrote: We currently don't use strict for plugins. Anyone object to me making it so? Did you just add it to the wrapper code to see whether it would break anything (at least for the most common plugins)? I just did

Re: perl switches on qpsmtpd? (was Re: Should we make plugins strict)

2005-05-25 Thread Matt Sergeant
On 25 May 2005, at 11:37, Elliot F wrote: Speaking of which (sort of), it looks like the high_perf branch does not have taint turned on.. Is this for performance reasons, or is this a choice? [EMAIL PROTECTED] $ /usr/bin/perl -Tw ./qpsmtpd Insecure dependency in require while running with

Re: Server load and the adaptive logging plugin

2005-05-24 Thread Matt Sergeant
On 24 May 2005, at 13:02, John Peacock wrote: Thoughts? To be honest John I missed all the adaptive logging stuff due to working on high_perf. I'm confused as to why you buffer up the log lines like this. Can you explain? Feel free to point me to a list archive entry if it's been explained

Re: Deep recursion with new logging and missing configs

2005-05-18 Thread Matt Sergeant
On 18 May 2005, at 17:52, John Peacock wrote: Now if only I could track down where these Plugins already loaded messages are coming from. It comes from load_plugins() being called in Qpsmtpd::TcpServer::run() for each connection. It's harmless (in that the plugins are only ever actually

Re: high_perf: Deep recursion on subroutine

2005-05-13 Thread Matt Sergeant
On 13 May 2005, at 15:33, Brian Grossman wrote: This caused a memory leak, so I've had to go back to one connection per accept_handler. If we use check_earlytalker, each new connection goes into EventLoop() for N seconds (via Client:can_read()), delaying the continuation of the accept_handler

Re: high_perf: Deep recursion on subroutine

2005-05-13 Thread Matt Sergeant
On 13 May 2005, at 16:12, Matt Sergeant wrote: I'm also wondering if there's a better way to do can_read than going back into the event loop... Correction to what I wrote in the previous email... There's no other way to do it than figure out the above... The good news is that if I can figure

Re: high_perf: Deep recursion on subroutine

2005-05-12 Thread Matt Sergeant
On 12 May 2005, at 17:28, Brian Grossman wrote: On Thu, 12 May 2005 20:06:26 + (UTC) [EMAIL PROTECTED] wrote: entirely. BTW, my biggest cpu load is when pollserver accepts connections. Ah, this was my fault. Get latest SVN. What was happening was I only accepted one connection per

Re: PATCH: high_perf: race condition with PostLoopCallback

2005-05-12 Thread Matt Sergeant
On 12 May 2005, at 17:35, Brian Grossman wrote: I have a fix for _load_plugins, but not peer_ip_string - I got around that slowness by turning off the max per IP code. Here's both then. I'll apply your peer_ip cache (using the key name peer_ip instead of rem_ip), but my patch for hook caching is

Re: PATCH: high_perf: race condition with PostLoopCallback

2005-05-12 Thread Matt Sergeant
On 12 May 2005, at 19:01, Brian Grossman wrote: Ah fair enough. I wonder if it wouldn't be faster to have an external hash of PLCs that gets cleared on DESTROY. Then we don't have to iterate through every single connection. That's definitely an improvement for me. Patch attached. Super.

Re: Problem while trying to implement TLS

2005-05-11 Thread Matt Sergeant
On 10 May 2005, at 20:33, Michael Holzt wrote: Because the current qpsmtpd implementation uses stdin and stdout for network communication i need to retransform stdin/stdout into an ssl-capable socket by using IO::Socket::SSL-new_from_fd. If qpsmtpd would directly use an IO::Socket::INET socket,

Re: PATCH: high_perf: race condition with PostLoopCallback

2005-05-11 Thread Matt Sergeant
On 11 May 2005, at 13:05, Brian Grossman wrote: The biggest slow down though was the per-object post-loop-callbacks. Iterating through each object every time through our event loop was a killer. I have a patch for that where I use an external variable to keep track of whether there are any PLCs

Re: high_perf: Deep recursion on subroutine

2005-05-11 Thread Matt Sergeant
On 11 May 2005, at 13:19, Brian Grossman wrote: Is anyone else who's using high_perf seeing error messages like this: Deep recursion on subroutine Qpsmtpd::run_hooks at \ .../lib/Qpsmtpd.pm line 54. For me, line 54 is the line in Qpsmptd::config() that calls run_hooks(config). I

Re: PATCH: high_perf: race condition with PostLoopCallback

2005-05-10 Thread Matt Sergeant
On 7 May 2005, at 18:08, Brian Grossman wrote: PostLoopCallback was a global and would be overwritten by anyone with an interest. This caused a race condition. For example, check_earlytalker would fail horribly if there was more than one connection. Although I applied this patch I'm confused

Re: non-blocking database/ldap/whatever plugins for high_perf branch?

2005-05-10 Thread Matt Sergeant
On 10 May 2005, at 16:21, Elliot F wrote: I'm looking at moving to the high_perf branch, or at least doing dev on it, as it looks promising and educational. I have used some of my plugins for normal qpsmtpd with high_perf, but blocking occurs. While this isn't an issue for a smaller server, I

Re: PATCH: make ConfigServer's pause and continue commands work

2005-05-09 Thread Matt Sergeant
Yeah I'm not so sure about the workings I have of pause/continue. There's a major issue in that if you pause/quit/telnet/continue then it'll never start again! On 5 May 2005, at 23:58, Brian Grossman wrote: Here's a patch to make the high_perf ConfigServer's PAUSE and CONTINUE commands work.

Re: Segmentation faults with qpsmtpd-forkserver?

2005-05-05 Thread Matt Sergeant
On 4 May 2005, at 18:22, Brian Grossman wrote: On Wed, 4 May 2005 16:22:53 -0400 Matt Sergeant [EMAIL PROTECTED] wrote: I'd love to hear of a larger scale rollout of high_perf. We've been pretty happy with it so far (except for ip_conntrack tables filling up - if anyone knows why that is - I

Re: Segmentation faults with qpsmtpd-forkserver?

2005-05-04 Thread Matt Sergeant
On 4 May 2005, at 10:45, John Peacock wrote: This would seem to suggest that you should be running the high-perf branch if possible (but I don't think the AV plugins have been re-implemented on that branch yet). The AV plugins don't need converting since they use system() so we can't do

Re: Segmentation faults with qpsmtpd-forkserver?

2005-05-04 Thread Matt Sergeant
On 3 May 2005, at 13:28, Justin Erenkrantz wrote: We've been seeing a spike in segmentation faults with qpsmtpd-forkserver in the last few days (nothing changed on our end). Has anyone else seen this behavior? Our master qpsmtpd perl process segfaults (i.e. not the children; but the parent) -

Re: Segmentation faults with qpsmtpd-forkserver?

2005-05-04 Thread Matt Sergeant
On 4 May 2005, at 17:16, Justin Erenkrantz wrote: --On Wednesday, May 4, 2005 5:05 PM -0400 Matt Sergeant [EMAIL PROTECTED] wrote: Time to switch to Apache::Qpsmtpd. Seriously? My impression from this list was that Apache::Qpsmtpd was largely untested. Has that changed? -- justin I believe

Re: [sqlite] DBD-SQLite build query

2005-04-29 Thread Matt Sergeant
On 28 Apr 2005, at 17:53, Clark Christensen wrote: For what it's worth, it looks like getsqlite.pl hasn't been updated in quite some time. It gets the package just fine, but it extracts the archive using the archive's embedded dirnames, then changes directory to 'sqlite', at which point, the rest

Re: [sqlite] DBD-SQLite build query

2005-04-29 Thread Matt Sergeant
On 28 Apr 2005, at 17:53, Clark Christensen wrote: For what it's worth, it looks like getsqlite.pl hasn't been updated in quite some time. It gets the package just fine, but it extracts the archive using the archive's embedded dirnames, then changes directory to 'sqlite', at which point, the rest

Now in ConfigServer

2005-04-28 Thread Matt Sergeant
LIST - lists current sessions sorted by time connected, with a limit if required (specify 10 to get oldest 10, specify -10 to get newest 10). KILL - kill a session STATUS - dumps the following style info: Current Status as of Fri Apr 29 02:10:20 2005 GMT Uptime: 22746.75 sec Mails

Re: Migrating to SVN

2005-04-27 Thread Matt Sergeant
On 27 Apr 2005, at 02:06, Tom Schindl wrote: Although I can not vote I would second this. I would also suggest to register for the issue/bug tracking one of the apache possibilities JIRA looks really cool although written in java. I haven't looked at scarab until now but could take a look.

Re: New axkit?

2005-04-27 Thread Matt Sergeant
On 26 Apr 2005, at 21:33, Michael Nachbaur wrote: On Apr 26, 2005, at 11:57 AM, Matt Sergeant wrote: OK guys, I think we should put out a new axkit release. There are lots of reports that CVS is stable, and even if there are things we should fix we can do them in 1.6.4 (or whatever the next

Re: Migrating to SVN

2005-04-27 Thread Matt Sergeant
On 27 Apr 2005, at 10:30, Tom Schindl wrote: Now there would be a perfect point to move AxKit into such a Bug/Feature tracking system. Bugs could be scheduled, ... . I've talked to Dirk about the possibility of us using Trac, which seems to integrate lots of project management features very

New axkit?

2005-04-26 Thread Matt Sergeant
OK guys, I think we should put out a new axkit release. There are lots of reports that CVS is stable, and even if there are things we should fix we can do them in 1.6.4 (or whatever the next number we choose is) afterwards. Any objections or comments? I'll wait until the weekend for votes.

Migrating to SVN

2005-04-26 Thread Matt Sergeant
Proposal: After we release 1.6.3, we migrate to subversion. Apparently it's a very simple process, and I've used svn on a couple of projects now and I'm happy with its stability and performance. Votes please! +1

Rudimentary config server

2005-04-26 Thread Matt Sergeant
I've just added a rudimentary config server to the high_perf branch. This allows you to connect on localhost to check number of concurrent connections and the number of pending dns queries. I have a few thoughts on other things that can be done, but I'd also like feedback on what people would

Re: Rudimentary config server

2005-04-26 Thread Matt Sergeant
On 26 Apr 2005, at 13:50, Bob wrote: Pause/Continue, not so simple. I can appreciate that there would be lots of state issues to handle to deliver a simple sounding feature. Well it would be more like stop accepting connections and then resume accepting. It's really trivial to do that :-) Matt.

Re: Rudimentary config server

2005-04-26 Thread Matt Sergeant
On 26 Apr 2005, at 16:56, Brian Grossman wrote: Erm, could you point one out to me for an example? I'm hoping in particular to implement booting a connection when it doesn't say anything for several minutes. I do this in forkserver with SIG{ALRM}, for example. It just does it. It times out

Re: uridnsbl: bogus rr run ...

2005-04-26 Thread Matt Sergeant
On 25 Apr 2005, at 20:18, Sidney Markowitz wrote: Matt Sergeant wrote: May be a problem with forking. Here's part of the fork replacement I use in my code that uses the single-packet-DNS stuff: Justin's code generates a number from the pid to initialize the ID counter and keeps track of it itself

Re: ESQL and MD5

2005-04-25 Thread Matt Sergeant
On 25 Apr 2005, at 08:00, [EMAIL PROTECTED] wrote: But it always fails. I tried to test the md5 taglib by just doing md5:md5testphrase/md5:md5 on a blank xsp page and it gives the result '8?u???}4'. This really doesn't look like a proper MD5 result. SO, where am I going wrong!? That tag

Re: uridnsbl: bogus rr run ...

2005-04-25 Thread Matt Sergeant
On 24 Apr 2005, at 17:01, Sidney Markowitz wrote: This could happen if the random ID isn't random enough May be a problem with forking. Here's part of the fork replacement I use in my code that uses the single-packet-DNS stuff: sub _fork { my $pid = fork; if (!defined($pid)) { die Cannot

Re: [Bug 3997] DNS answers get mixed up

2005-04-13 Thread Matt Sergeant
On 13 Apr 2005, at 09:00, [EMAIL PROTECTED] wrote: --- Additional Comments From [EMAIL PROTECTED] 2005-04-13 06:00 --- Tony, yes, now that we are using the DNS ID to match up replies to queries there is no reason to use more than one socket for that purpose. Where was your

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