Done in r1517478. On Mon, Aug 26, 2013 at 1:27 PM, Stefan Sperling <[email protected]> wrote: > On Mon, Aug 26, 2013 at 09:15:45AM -0000, [email protected] wrote: >> Author: ivan >> Date: Mon Aug 26 09:15:44 2013 >> New Revision: 1517472 >> >> URL: http://svn.apache.org/r1517472 >> Log: >> ra_serf: Remove DAV: headers from GET requests. This decrease outbound >> traffic during checkout/update by 45%. >> >> * subversion/libsvn_ra_serf/ra_serf.h >> (svn_ra_serf__handler_t): Add NO_DAV_HEADERS member. >> >> * subversion/libsvn_ra_serf/update.c >> (fetch_file, svn_ra_serf__get_file): Set NO_DAV_HEADERS=TRUE for GET >> request. >> >> * subversion/libsvn_ra_serf/util.c >> (setup_serf_req): Add DAV: request headers only if DAV_HEADERS parameter >> is non-zero. >> (setup_request): Pass ! NO_DAV_HEADERS to setup_serf_req() call. > >> @@ -786,9 +790,12 @@ setup_serf_req(serf_request_t *request, >> /* These headers need to be sent with every request; see issue #3255 >> ("mod_dav_svn does not pass client capabilities to start-commit >> hooks") for why. */ > > Can you please update the above comment to justify the exception > made in case dav_headers is FALSE? > >> - serf_bucket_headers_setn(*hdrs_bkt, "DAV", SVN_DAV_NS_DAV_SVN_DEPTH); >> - serf_bucket_headers_setn(*hdrs_bkt, "DAV", SVN_DAV_NS_DAV_SVN_MERGEINFO); >> - serf_bucket_headers_setn(*hdrs_bkt, "DAV", >> SVN_DAV_NS_DAV_SVN_LOG_REVPROPS); >> + if (dav_headers) >> + { >> + serf_bucket_headers_setn(*hdrs_bkt, "DAV", SVN_DAV_NS_DAV_SVN_DEPTH); >> + serf_bucket_headers_setn(*hdrs_bkt, "DAV", >> SVN_DAV_NS_DAV_SVN_MERGEINFO); >> + serf_bucket_headers_setn(*hdrs_bkt, "DAV", >> SVN_DAV_NS_DAV_SVN_LOG_REVPROPS); >> + } >> >> return SVN_NO_ERROR; >> }
-- Ivan Zhakov CTO | VisualSVN | http://www.visualsvn.com
