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.

<snip>
127.0.0.1 - jrandom [07/Dec/2012:17:26:57 +0530] "REPORT http://localhost:25001/svn-test-work/repositories/svnsync_tests-1 HTTP/1.1" 200 303
</snip>

After applying this patch, it will look like,

<snip>
127.0.0.1 - jrandom [07/Dec/2012:17:24:21 +0530] "REPORT /svn-test-work/repositories/svnsync_tests-1 HTTP/1.1" 200 303
</snip>

Attached the patch and log message.

Thanks & Regards,
Vijayaguru
Index: subversion/libsvn_ra_serf/replay.c
===================================================================
--- subversion/libsvn_ra_serf/replay.c  (revision 1418178)
+++ subversion/libsvn_ra_serf/replay.c  (working copy)
@@ -655,7 +655,7 @@
 
   handler->handler_pool = pool;
   handler->method = "REPORT";
-  handler->path = session->session_url_str;
+  handler->path = session->session_url.path;
   handler->body_delegate = create_replay_body;
   handler->body_delegate_baton = replay_ctx;
   handler->body_type = "text/xml";
@@ -798,7 +798,7 @@
 
           handler->handler_pool = replay_ctx->src_rev_pool;
           handler->method = "REPORT";
-          handler->path = session->session_url_str;
+          handler->path = session->session_url.path;
           handler->body_delegate = create_replay_body;
           handler->body_delegate_baton = replay_ctx;
           handler->conn = session->conns[0];
* 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.

Patch by: Vijayaguru G <vijay{_AT_}collab.net>                            

Reply via email to