Re: [PATCH] Replay report logs the entire session url in httpd access log

Philip Martin <philip.martin_at_wandisco.com>
> vijay <vijay_at_collab.net> writes:
>>  Attached the patch and log message.
>
> Committed in r1418322. Thanks!

Hi Philip and Vijay.

Your log message:
[[[
* subversion/libsvn_ra_serf/replay.c
  (svn_ra_serf__replay, svn_ra_serf__replay_range): Replace the
    session url string with the request path portion of the url.
]]]

leaves me thinking, "But *why* did you replace the session url string with the request path portion of the url?"

Please edit the log message to say why this change was made and what it means in functional terms (like, what behaviour does it change, does it fix a bug, etc.).

- Julian
Received on 2012-12-07 15:19:18 CET