Re: [PATCH] Unknown cfg function

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