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

From: Amos Jeffries <squid3_at_treenet.co.nz>
Date: Sat, 30 Jun 2012 14:06:57 +1200

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
Received on Sat Jun 30 2012 - 02:07:33 MDT

This archive was generated by hypermail 2.2.0 : Sat Jun 30 2012 - 12:00:06 MDT