On Thu, 14 Feb 2008 15:07:38 -0500
Jim Jagielski <[EMAIL PROTECTED]> wrote:

> Right now, both of these are defined as
> APR_IMPLEMENT_EXTERNAL_HOOK_RUN_FIRST and the ones we use
> are at APR_HOOK_FIRST. They also always return OK (unless
> an error) so this means that we can never add additional
> hooks here and have them run (don't talk to me about
> REALLY_FIRST :) ).
> 
> Anyway, considering some of the LB stuff we've been talking
> about, it would be very useful to be able to have providers
> and others utilize these hooks (eg: for timestamping start
> and end of requests), but with RUN_FIRST, we're limiting
> that.
> 
> Soooooo I'm proposing changing to RUN_ALL...
> 
> Comments?

That's a public API you're proposing to change.  Ergo, something
on a 2.4 timescale.

Wouldn't changing existing HOOK_FIRST functions to return DECLINED
achieve what you need without breaking anything?

-- 
Nick Kew

Application Development with Apache - the Apache Modules Book
http://www.apachetutor.org/

Reply via email to