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
