sterling wrote:

> That would work IFF reset_filters guarentees that there is only one
> instance of each of those filters, it looks to me like reset_filters does
> not provide that functionality (it just removes all filters not named
> {CORE,CONTENT_LENGTH,HTTP_HEADER}).

You're right (switches brain on manually, it's late and the weekend was
long).

> Even if it did, is that the desired functionality? (i.e. on error remove
> all filters except those required by the core).  I would expect we just
> want a function to ensure the request has those core filters - and
> leave all other inserted filters.  Then ap_die could call that
> function.

Maybe it should just check whether the core filters are there, and add
them if not.

Regards,
Graham
-- 
-----------------------------------------
[EMAIL PROTECTED]                "There's a moon
                                        over Bourbon Street
                                                tonight..."

S/MIME Cryptographic Signature

Reply via email to