Re: [PATCH] URL logformat codes %<ru and %>ru

 On Wed, 18 May 2011 10:04:11 -0600, Alex Rousskov wrote:
> On 05/18/2011 08:52 AM, Amos Jeffries wrote:
>> This patch adds the logging codes:
>> %>ru - Request URL from the client original request.
>> %<ru - Request URL sent to the server.
>
> It also fixes or changes %rp behavior to use adapted URI, if any.
> Please
> mention that in the commit message if that fix (or change) is
> intentional. If the change makes %rp always behave like %<rp (path
> sent
> to the server), perhaps squid.conf should be updated accordingly as
> well?
>
>
>> Ideally these will log the original syntax, but Squid does not
>> preserve
>> that. For now the canonical URI are used instead.
>>
>> The %ru is retained to display the same old historic logging URL it
>> used
>> to. This appears to be the client original in most cases, but has
>> been
>> observed with different content to both the canonical client URL and
>> canonical server URL in some cases.
>
> Perhaps we should leave %rp to have the same inconsistent behavior as
> %ru then? It is kind of wrong if the two get out of sync. Adding %<rp
> and %>rp would be fine, of course.

 Oops, yes that was the intention. Fixing by adding the other codes.

 Amos
Received on Wed May 18 2011 - 23:48:20 MDT