Re: [PATCH] Unknown cfg function

From: Amos Jeffries <squid3_at_treenet.co.nz>
Date: Tue, 30 Jul 2013 12:31:01 +1200

On 30/07/2013 5:25 a.m., Amos Jeffries wrote:
> Looks like nobody actually ran the trivial "-k parse" test using
> 3.HEAD or the initial quoted strings code.
>
> Attached is a patch which adds a secondary form of undo for handling
> incorrectly identified ConfigParser::FunctionNameToken elements.
> Instead of aborting Squid on any non-function token containing a '('
> we take the wrongly identified assumed function name element and
> reform it with the '(' delimiter and trailing element. The resulting
> aggregate token which should have been identified initially is then
> sent to the upper parser layer.
>
> Amos
>

Updated patch. This one does all the above plus:
* handles the case where a token starts with '(' as in the default
refresh_pattern for cgi-bin handling.
* removes $macro handling, resolving the case where $ symbols in regex
patterns are placed inside "-quoted strings
* adds debug statemets for NextToken response outputs
* adds debugs statements for parameters("foo") file loading

Amos

Received on Tue Jul 30 2013 - 00:31:07 MDT

This archive was generated by hypermail 2.2.0 : Tue Jul 30 2013 - 12:00:50 MDT