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

From: Amos Jeffries <squid3_at_treenet.co.nz>
Date: Thu, 12 Jul 2012 11:47:11 +1200

On 12.07.2012 06:49, Tsantilas Christos wrote:
> This is the new version of the request_header_add patch. It is a
> little
> bigger than the original it make many changes, but I hope it is OK
> and
> did not add bugs to squid.
>
> This patch:
> - The format/* interface used to format added headers.
> - The logformat codes used for request_header_add option
> - Add the new %ssl::>cert_subject and %ssl::>cert_issuer log
> formating
> codes
> - The request_header_add project has as requirements login,client
> ip
> address and the server name. The %ul, %>a and %{Host}>h can be used
> with
> this patch.
> - AccessLogEntry is a RefCountable class in this patch.
>
> I hope it is OK.

+1. Looks okay to me.

Would you be able to commit the AccessLogEntry and libformat changes
separate from the header mangling please. I'm happy for both to go in at
once though.

Amos
Received on Wed Jul 11 2012 - 23:47:19 MDT

This archive was generated by hypermail 2.2.0 : Tue Jul 17 2012 - 12:00:03 MDT