[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Fri, 7 Dec 2012 14:18:42 +0000 (GMT)

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

This is an archived mail posted to the Subversion Dev mailing list.