You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by sv...@apache.org on 2012/10/22 06:03:36 UTC
svn commit: r1400757 - in /subversion/branches/1.7.x: ./ STATUS
subversion/mod_dav_svn/repos.c
Author: svn-role
Date: Mon Oct 22 04:03:35 2012
New Revision: 1400757
URL: http://svn.apache.org/viewvc?rev=1400757&view=rev
Log:
Merge r1390653 from trunk:
* r1390653
Add Vary: header to GET response.
Justification:
The 1.8 serf-only client may fail to work with caching proxies.
Votes:
+1: philip, cmpilato, breser
+0: danielsh
Modified:
subversion/branches/1.7.x/ (props changed)
subversion/branches/1.7.x/STATUS
subversion/branches/1.7.x/subversion/mod_dav_svn/repos.c
Propchange: subversion/branches/1.7.x/
------------------------------------------------------------------------------
Merged /subversion/trunk:r1390653
Modified: subversion/branches/1.7.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x/STATUS?rev=1400757&r1=1400756&r2=1400757&view=diff
==============================================================================
--- subversion/branches/1.7.x/STATUS (original)
+++ subversion/branches/1.7.x/STATUS Mon Oct 22 04:03:35 2012
@@ -75,11 +75,3 @@ Veto-blocked changes:
Approved changes:
=================
-
- * r1390653
- Add Vary: header to GET response.
- Justification:
- The 1.8 serf-only client may fail to work with caching proxies.
- Votes:
- +1: philip, cmpilato, breser
- +0: danielsh
Modified: subversion/branches/1.7.x/subversion/mod_dav_svn/repos.c
URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x/subversion/mod_dav_svn/repos.c?rev=1400757&r1=1400756&r2=1400757&view=diff
==============================================================================
--- subversion/branches/1.7.x/subversion/mod_dav_svn/repos.c (original)
+++ subversion/branches/1.7.x/subversion/mod_dav_svn/repos.c Mon Oct 22 04:03:35 2012
@@ -3075,6 +3075,13 @@ set_headers(request_rec *r, const dav_re
if ((serr == NULL) && (info.rev != SVN_INVALID_REVNUM))
{
mimetype = SVN_SVNDIFF_MIME_TYPE;
+
+ /* Note the base that this svndiff is based on, and tell any
+ intermediate caching proxies that this header is
+ significant. */
+ apr_table_setn(r->headers_out, "Vary", SVN_DAV_DELTA_BASE_HEADER);
+ apr_table_setn(r->headers_out, SVN_DAV_DELTA_BASE_HEADER,
+ resource->info->delta_base);
}
svn_error_clear(serr);
}