Re: [PATCH] Add request_header_add option and [request|reply]_header_* manglers fixes

From: Tsantilas Christos <chtsanti_at_users.sourceforge.net>
Date: Thu, 05 Jul 2012 17:42:33 +0300

On 06/30/2012 05:06 AM, Amos Jeffries wrote:
> On 30/06/2012 5:28 a.m., Tsantilas Christos wrote:
>> I split the initial patch to two patches. The request_header_manglers
>> patch should applied first.
>>
> <snip>
>>> * please make a new header file for the new class HeaderManglers and
>>> HeadersWithAcl. We are trying to remove structs.h file.
>> OK. I create the header file HttpHeaderTools.h
>> Maybe we need to move HeadersWithAcls and HeaderManglers to structs.h
>> again, to be applied to squid-3.2
>
> That should be fine.
>
> +1 for commit of custom/other header access/replace if you are both
> happy with it.
>
> Amos
>
>

The request_header_manglers-v6.1.patch applied to trunk.
Received on Thu Jul 05 2012 - 14:42:45 MDT

This archive was generated by hypermail 2.2.0 : Thu Jul 05 2012 - 12:00:03 MDT