On Wed, Jan 10, 2018 at 7:13 AM, Yann Ylavic <ylavic....@gmail.com> wrote:
> On Wed, Jan 10, 2018 at 1:52 AM,  <cove...@apache.org> wrote:
>> Author: covener
>> Date: Wed Jan 10 00:52:25 2018
>> New Revision: 1820715
>>
>> URL: http://svn.apache.org/viewvc?rev=1820715&view=rev
>> Log:
>> PR 61983: "Header unset Content-Type" doesn't work
>>
>>
>> Submitted By: Hank Ibell <hwibell gmail.com>
>> Committed By: covener
> []
>>
>> Modified: httpd/httpd/trunk/modules/metadata/mod_headers.c
>> URL: 
>> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/metadata/mod_headers.c?rev=1820715&r1=1820714&r2=1820715&view=diff
>> ==============================================================================
>> --- httpd/httpd/trunk/modules/metadata/mod_headers.c (original)
>> +++ httpd/httpd/trunk/modules/metadata/mod_headers.c Wed Jan 10 00:52:25 2018
>> @@ -806,6 +806,9 @@ static int do_headers_fixup(request_rec
>>              break;
>>          case hdr_unset:
>>              apr_table_unset(headers, hdr->header);
>> +            if (!ap_cstr_casecmp(hdr->header, "Content-Type")) {
>> +                ap_set_content_type(r, NULL);
>> +            }
>
> Don't we need to also check for "headers != r->headers_in &&
> !ap_cstr_casecmp(...)" here?

I see what you mean, this would be in fixups (after type checker) and
we could clobber the outbound side. Thanks.

Reply via email to