On 1 Dec 2008, at 08:52, Ruediger Pluem wrote:

2. IMHO ap_invoke_handler is only called in three locations:

It's a public API, so could be called by any module.

So ap_die maybe should do the following:

    if (type == AP_FILTER_ERROR) {
        if [ap_http_header_filter is still in chain] {
           log this
           type = HTTP_INTERNAL_SERVER_ERROR;
        }
        else {
           return;
        }
    }

That looks good, too.

But do you see any objection to the (IMHO simpler) fix of
removing error buckets as we detect them?

--
Nick Kew

Reply via email to