Re: [PATCH] Bug 3389: Auto-reconnect for tcp access_log

From: Alex Rousskov <rousskov_at_measurement-factory.com>
Date: Mon, 29 Apr 2013 13:14:03 -0600

Hello,

    The attached r12779 patch contains all current cumulative changes
for bug 3389. Please see the patch preamble for technical details. They
have not changed since the last submission except I added (iii) to
clarify parsing assumption. The changes correspond to the following lp
branch (at revision 12779):
https://code.launchpad.net/~squid/squid/bug3389

This snapshot implements the following Amos request:

On 04/29/2013 08:59 AM, Amos Jeffries wrote:

> * I object to adding the "logger=" tag on the front of the module:place
> field. It seems pointless and a waste of letters.
> If the first token after module:place is not an option it must be teh
> format name old syntax) or an ACL name (new syntax).
> This should be easy enough to fix without adding a surprise logger=
> token to the directive.

The changes to implement the above are limited to src/cache_cf.cc and
src/cf.data.pre.

I believe this snapshot addresses all review issues raised so far.

HTH,

Alex.

Received on Mon Apr 29 2013 - 19:14:16 MDT

This archive was generated by hypermail 2.2.0 : Tue Apr 30 2013 - 12:00:09 MDT