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

2012-12-07 Thread vijay
Thanks, Philip and Julian. I will explain things clearly in future patches. Thanks & Regards, Vijayaguru On Friday 07 December 2012 10:02 PM, Julian Foad wrote: Philip Martin wrote: Philip Martin writes: The logging is really just a side-effect. We setup this structure in several pla

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

2012-12-07 Thread Julian Foad
Philip Martin wrote: > Philip Martin writes: >> The logging is really just a side-effect.  We setup this structure in >> several places and set the path member to a path.  In one place we put a >> full URL in by mistake.  As far as I can tell the member is supposed to >> be a path and the URL

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

2012-12-07 Thread Julian Foad
Philip Martin wrote: > Julian Foad writes: >>   The mod_dav_svn replay report was logging the entire session >>   URL in the httpd access log.  Make it do what it is supposed >>   to do, which is log just the path-within-repos. > > The logging is really just a side-effect.  We setup this stru

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

2012-12-07 Thread Philip Martin
Philip Martin writes: > Julian Foad writes: > >>   The mod_dav_svn replay report was logging the entire session >>   URL in the httpd access log.  Make it do what it is supposed >>   to do, which is log just the path-within-repos. > > The logging is really just a side-effect. We setup this stru

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

2012-12-07 Thread Philip Martin
Julian Foad writes: >   The mod_dav_svn replay report was logging the entire session >   URL in the httpd access log.  Make it do what it is supposed >   to do, which is log just the path-within-repos. The logging is really just a side-effect. We setup this structure in several places and set t

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

2012-12-07 Thread Julian Foad
Philip Martin wrote: > Julian Foad writes: >> Philip Martin >>> vijay 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,

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

2012-12-07 Thread Philip Martin
Julian Foad writes: > Philip Martin >> vijay 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 >    

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

2012-12-07 Thread Julian Foad
Philip Martin > vijay 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

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

2012-12-07 Thread Philip Martin
vijay writes: > Attached the patch and log message. Committed in r1418322. Thanks! -- Certified & Supported Apache Subversion Downloads: http://www.wandisco.com/subversion/download

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

2012-12-07 Thread vijay
Hi, The REPORT request during replay is being logged in httpd access log with the entire session url instead of the request path portion of the url. E.g, Now the REPORT request during 'svnsync sync' or 'svnrdump dump' is logged in httpd access log as follows. 127.0.0.1 - jrandom [07/Dec/2