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

Reply via email to