Re: New post-log-transaction hook?

2001-09-24 Thread Rodent of Unusual Size
"Roy T. Fielding" wrote: > > I think that anyone using "tripe" in an analogy should be > forced to eat it. Don't go there, Roy, or someone is sure to apply it to 'bullsh*t' as well.. :-D -- #kenP-)} Ken Coar, Sanagendamgagwedweinini http://Golux.Com/coar/ Author, developer, opinionist

Re: New post-log-transaction hook?

2001-09-21 Thread Roy T. Fielding
> Exactly, the client is still connected, but by this time, the client should have > received all of the data. The only thing we have is a connection that will be > left open until both sides get around to closing it. If a non-keepalive request > is still receiving data after the log-transaction

Re: New post-log-transaction hook?

2001-09-21 Thread Roy T. Fielding
On Wed, Sep 19, 2001 at 03:42:42PM -0400, Rodent of Unusual Size wrote: > Greg Stein wrote: > > > > p.s. "utter tripe" indeed... that was rather inflammatory... > > Sorry, but the whole thrust of your message seemed to be > 'cleanups can't depend on diddly-squat'. I didn't say it > *was* tripe,

[PATCH] fix cleanups in cleanups (Was Re: New post-log-transaction hook?)

2001-09-20 Thread Aaron Bannert
On Wed, Sep 19, 2001 at 12:27:35PM -0700, Jon Travis wrote: > BZzzzt. The attached code registers a cleanup from within a cleanup, and > does so 'correctly'. See the program attached at the bottom, which behaves > incorrectly. It is simple code, but not knowing that a given > function register

Re: pool cleanup (was: Re: New post-log-transaction hook?)

2001-09-20 Thread Ryan Bloom
On Wednesday 19 September 2001 02:21 pm, Greg Stein wrote: > On Wed, Sep 19, 2001 at 12:16:24PM -0700, Ryan Bloom wrote: > > On Wednesday 19 September 2001 11:37 am, William A. Rowe, Jr. wrote: > > > From: "Greg Stein" <[EMAIL PROTECTED]> > > > Sent: Wednesday, September 19, 2001 1:26 PM > > > Rea

pool cleanup (was: Re: New post-log-transaction hook?)

2001-09-19 Thread Greg Stein
On Wed, Sep 19, 2001 at 12:16:24PM -0700, Ryan Bloom wrote: > On Wednesday 19 September 2001 11:37 am, William A. Rowe, Jr. wrote: > > From: "Greg Stein" <[EMAIL PROTECTED]> > > Sent: Wednesday, September 19, 2001 1:26 PM >... > > > The problem is cross-dependency between the cleanup actions. One

Re: New post-log-transaction hook?

2001-09-19 Thread Rodent of Unusual Size
Greg Stein wrote: > > p.s. "utter tripe" indeed... that was rather inflammatory... Sorry, but the whole thrust of your message seemed to be 'cleanups can't depend on diddly-squat'. I didn't say it *was* tripe, just that it sounded like it. :-) -- #kenP-)} Ken Coar, Sanagendamgagwedweinini

Re: New post-log-transaction hook?

2001-09-19 Thread Ryan Bloom
On Wednesday 19 September 2001 12:27 pm, Jon Travis wrote: > On Wed, Sep 19, 2001 at 12:16:24PM -0700, Ryan Bloom wrote: > > On Wednesday 19 September 2001 11:37 am, William A. Rowe, Jr. wrote: > > > From: "Greg Stein" <[EMAIL PROTECTED]> > > > Sent: Wednesday, September 19, 2001 1:26 PM > > > > >

Re: New post-log-transaction hook?

2001-09-19 Thread Jon Travis
On Wed, Sep 19, 2001 at 12:16:24PM -0700, Ryan Bloom wrote: > On Wednesday 19 September 2001 11:37 am, William A. Rowe, Jr. wrote: > > From: "Greg Stein" <[EMAIL PROTECTED]> > > Sent: Wednesday, September 19, 2001 1:26 PM > > > > > On Wed, Sep 19, 2001 at 01:52:12PM -0400, Rodent of Unusual Size w

Re: New post-log-transaction hook?

2001-09-19 Thread Ryan Bloom
On Wednesday 19 September 2001 11:37 am, William A. Rowe, Jr. wrote: > From: "Greg Stein" <[EMAIL PROTECTED]> > Sent: Wednesday, September 19, 2001 1:26 PM > > > On Wed, Sep 19, 2001 at 01:52:12PM -0400, Rodent of Unusual Size wrote: > > > Greg Stein wrote: > > > > It isn't a bug. Cleanups are for

Re: New post-log-transaction hook?

2001-09-19 Thread William A. Rowe, Jr.
From: "Greg Stein" <[EMAIL PROTECTED]> Sent: Wednesday, September 19, 2001 1:26 PM > On Wed, Sep 19, 2001 at 01:52:12PM -0400, Rodent of Unusual Size wrote: > > Greg Stein wrote: > > > It isn't a bug. Cleanups are for just wrapping things up, > > > not doing work. > > > > If that's the authorit

Re: New post-log-transaction hook?

2001-09-19 Thread Greg Stein
On Wed, Sep 19, 2001 at 01:52:12PM -0400, Rodent of Unusual Size wrote: > Greg Stein wrote: > > It isn't a bug. Cleanups are for just wrapping things up, > > not doing work. > > If that's the authoritative answer, then we need to provide > a supported way for 'doing work' at cleanup time. > > >

Re: New post-log-transaction hook?

2001-09-19 Thread Rodent of Unusual Size
Greg Stein wrote: > > It isn't a bug. Cleanups are for just wrapping things up, > not doing work. If that's the authoritative answer, then we need to provide a supported way for 'doing work' at cleanup time. > You might not even be able to open and use that file if > your pool is n the process

Re: New post-log-transaction hook?

2001-09-19 Thread Greg Stein
On Wed, Sep 19, 2001 at 11:17:36AM -0500, William A. Rowe, Jr. wrote: > From: "Ryan Bloom" <[EMAIL PROTECTED]> > Sent: Wednesday, September 19, 2001 9:52 AM > > > > On Tuesday 18 September 2001 06:09 pm, Greg Stein wrote: > > > I agree with OtherBill. > > > > > > Cleanups are not always the answ

Re: New post-log-transaction hook?

2001-09-19 Thread William A. Rowe, Jr.
From: "Ryan Bloom" <[EMAIL PROTECTED]> Sent: Wednesday, September 19, 2001 9:52 AM > On Tuesday 18 September 2001 06:09 pm, Greg Stein wrote: > > I agree with OtherBill. > > > > Cleanups are not always the answer. When they are run, many things > > associated with that pool could be torn down al

Re: New post-log-transaction hook?

2001-09-19 Thread Ryan Bloom
On Tuesday 18 September 2001 06:09 pm, Greg Stein wrote: > I agree with OtherBill. > > Cleanups are not always the answer. When they are run, many things > associated with that pool could be torn down already because their cleanups > have already run. > > If you need a known state to perform *oper

Re: New post-log-transaction hook?

2001-09-18 Thread Greg Stein
On Tue, Sep 18, 2001 at 06:52:35AM -0400, Jeff Trawick wrote: > Greg Stein <[EMAIL PROTECTED]> writes: > > > 2) move the ap_lingering_close inside ap_process_connection, then call it > >from with ap_process_connection. This *almost* works. All MPMs have a > >call to ap_process_connection

Re: New post-log-transaction hook?

2001-09-18 Thread Greg Stein
I agree with OtherBill. Cleanups are not always the answer. When they are run, many things associated with that pool could be torn down already because their cleanups have already run. If you need a known state to perform *operations* (as it sounds like Jon is doing), then you can't use a cleanu

Re: New post-log-transaction hook?

2001-09-18 Thread Jon Travis
On Tue, Sep 18, 2001 at 02:20:35PM -0500, William A. Rowe, Jr. wrote: > From: "Ryan Bloom" <[EMAIL PROTECTED]> > Sent: Tuesday, September 18, 2001 11:44 AM > > > > On Tuesday 18 September 2001 08:17 am, William A. Rowe, Jr. wrote: > > > Why not let the MPM register the lingerclose with APR_HOOK_

Re: New post-log-transaction hook?

2001-09-18 Thread William A. Rowe, Jr.
From: "Ryan Bloom" <[EMAIL PROTECTED]> Sent: Tuesday, September 18, 2001 11:44 AM > On Tuesday 18 September 2001 08:17 am, William A. Rowe, Jr. wrote: > > Why not let the MPM register the lingerclose with APR_HOOK_MIDDLE in the > > post_connection hook? That way, if Jon's (or any other author's

Re: New post-log-transaction hook?

2001-09-18 Thread Ryan Bloom
On Tuesday 18 September 2001 08:17 am, William A. Rowe, Jr. wrote: > Why not let the MPM register the lingerclose with APR_HOOK_MIDDLE in the > post_connection hook? That way, if Jon's (or any other author's) intent is > to work before the lingering close, then it can be APR_HOOK_FIRST. > Otherw

Re: child_exit/pchild cleanup (was Re: New post-log-transaction hook?)

2001-09-18 Thread Ryan Bloom
On Tuesday 18 September 2001 08:19 am, Cliff Woolley wrote: > On Tue, 18 Sep 2001, Ryan Bloom wrote: > > > You've confused the issue with your subject line (everybody is bugging > > > out because they're relating it to logging). It should not have > > > anything to do with "log". We have a pre-con

Re: New post-log-transaction hook?

2001-09-18 Thread William A. Rowe, Jr.
al Message - From: "Jeff Trawick" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Tuesday, September 18, 2001 5:52 AM Subject: Re: New post-log-transaction hook? > Greg Stein <[EMAIL PROTECTED]> writes: > > > 2) move the ap_lingering_close inside ap

child_exit/pchild cleanup (was Re: New post-log-transaction hook?)

2001-09-18 Thread Cliff Woolley
On Tue, 18 Sep 2001, Ryan Bloom wrote: > > You've confused the issue with your subject line (everybody is bugging out > > because they're relating it to logging). It should not have anything to do > > with "log". We have a pre-connection hook, so call yours post-connection. > > That is when you w

Re: New post-log-transaction hook?

2001-09-18 Thread Ryan Bloom
On Tuesday 18 September 2001 02:10 am, Greg Stein wrote: > On Mon, Sep 17, 2001 at 03:52:21PM -0700, Jon Travis wrote: > > I've got a bit of code that needs to run after a connection to a client > > has been closed. Right now I can (kind of) spoof this by setting the > > keepalive for the client

Re: New post-log-transaction hook?

2001-09-18 Thread Ryan Bloom
On Monday 17 September 2001 09:33 pm, William A. Rowe, Jr. wrote: > From: "Jon Travis" <[EMAIL PROTECTED]> > Sent: Monday, September 17, 2001 6:32 PM > > > I tried setting keepalive == 0 in the handler, and doing my ju-ju in > > the log_transaction phase. The client was still hanging around. > >

Re: New post-log-transaction hook?

2001-09-18 Thread Jeff Trawick
Greg Stein <[EMAIL PROTECTED]> writes: > 2) move the ap_lingering_close inside ap_process_connection, then call it >from with ap_process_connection. This *almost* works. All MPMs have a >call to ap_process_connection followed by a call to ap_lingering_close. >The only MPM that does ot

Re: New post-log-transaction hook?

2001-09-18 Thread Greg Stein
On Mon, Sep 17, 2001 at 03:52:21PM -0700, Jon Travis wrote: > I've got a bit of code that needs to run after a connection to a client > has been closed. Right now I can (kind of) spoof this by setting the > keepalive for the client to 0, and registering a cleanup on the > request_req pool. Unfo

Re: New post-log-transaction hook?

2001-09-17 Thread William A. Rowe, Jr.
From: "Jon Travis" <[EMAIL PROTECTED]> Sent: Monday, September 17, 2001 6:32 PM > I tried setting keepalive == 0 in the handler, and doing my ju-ju in > the log_transaction phase. The client was still hanging around. That sounds right ... the lazy disconnect logic in httpd can leave a connect

Re: New post-log-transaction hook?

2001-09-17 Thread Jon Travis
I tried setting keepalive == 0 in the handler, and doing my ju-ju in the log_transaction phase. The client was still hanging around. -- Jon On Mon, Sep 17, 2001 at 04:11:58PM -0700, Ryan Bloom wrote: > On Monday 17 September 2001 03:52 pm, Jon Travis wrote: > > Why can't you do it in the log

RE: New post-log-transaction hook?

2001-09-17 Thread MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1)
September 17, 2001 4:12 PM To: [EMAIL PROTECTED]; Jon Travis Subject: Re: New post-log-transaction hook? On Monday 17 September 2001 03:52 pm, Jon Travis wrote: Why can't you do it in the log_transaction phase. Assuming this is not a keepalive connection, the client will be gone by the ti

Re: New post-log-transaction hook?

2001-09-17 Thread Ryan Bloom
On Monday 17 September 2001 03:52 pm, Jon Travis wrote: Why can't you do it in the log_transaction phase. Assuming this is not a keepalive connection, the client will be gone by the time that phase is run. If this is a keep-alive transaction, then you won't save anything by adding another phase

Re: New post-log-transaction hook?

2001-09-17 Thread Jon Travis
On Mon, Sep 17, 2001 at 07:01:21PM -0400, Cliff Woolley wrote: > On Mon, 17 Sep 2001, Jon Travis wrote: > > > I've got a bit of code that needs to run after a connection to a client > > has been closed. Right now I can (kind of) spoof this by setting the > > keepalive for the client to 0, and re

Re: New post-log-transaction hook?

2001-09-17 Thread Cliff Woolley
On Mon, 17 Sep 2001, Jon Travis wrote: > I've got a bit of code that needs to run after a connection to a client > has been closed. Right now I can (kind of) spoof this by setting the > keepalive for the client to 0, and registering a cleanup on the > request_req pool. Unfortunately the code in