On 29 May 2013, at 14:28, Thomas Eckert wrote:
> I saw the following
>
> /* only act if starts-with "text/" or contains "xml" */
> if (strncmp(ctype, "text/", 5) && !strstr(ctype, "xml")) {
> ap_remove_output_filter(f);
> return ap_pass_brigade(f->next, bb) ;
> }
You're right, that's a bug.
I don't think your patch really works either: any hardwired rules are
going to be susceptible to similar issues. I guess what it needs is to
be user-configurable, perhaps with HTML and XHTML as defaults.
--
Nick Kew
