httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Manoj Kasichainula <>
Subject Re: [PATCH] ap_add_filter
Date Sun, 27 Aug 2000 12:19:38 GMT
On Mon, Aug 21, 2000 at 10:39:02PM -0700, Manoj Kasichainula wrote:
> On Sun, Aug 20, 2000 at 06:29:11PM -0700, wrote:
> > I would really like to see this design in code, because even with all of
> > the discussion about it, I don't actually see how some of the details
> > work.  In my mind, this design is more complex, but Manoj, you keep saying
> > this is simple.  Please provide some compilable code, and enlighten me.
> It's coming (though I'd be pleased to see someone else beat me to it).
> I'd like to finish suexec first (which I don't think is far off).
> It won't be interesting code, though, because I'll end up modifying
> one of the existing filters that doesn't really need to add filters.
> But the concept should still be clear.

OK, the filter chain management stuff doesn't look anything like I
expected it to. I see something that looks kind of funky to me, with
the module adding a hook so that it can add itself to the filter
chain, and also registering itself with the core seperately.

Why two seperate calls?

I'm still feeling my way through this stuff, but I probably would have
had each module return an ap_filter_t pointer in its filtering hook
instead, and let the core engine actually add the filter to the filter
chain. No coincedence that this "goes with" my filter chain management
proposal :)

View raw message