You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@subversion.apache.org by ko...@apache.org on 2015/11/26 12:01:14 UTC

svn propchange: r1716593 - svn:log

Author: kotkov
Revision: 1716593
Modified property: svn:log

Modified: svn:log at Thu Nov 26 11:01:14 2015
------------------------------------------------------------------------------
--- svn:log (original)
+++ svn:log Thu Nov 26 11:01:14 2015
@@ -1,5 +1,6 @@
 Fixup the recent solution for issue SVN-4514 by discouraging caching for
-requests that specify ?r=WORKINGREV in the url, e.g.
+requests that include ?r=WORKINGREV, but don't contain ?p=PEGREV in the
+url, e.g.:
 
   https://svn.apache.org/repos/asf/subversion/trunk/README?r=1716593
 
@@ -9,6 +10,13 @@ history to the canonical ?p=PEGREV locat
 state of the repository, and cannot be cached, so make sure we set the
 "Cache-Control: max-age=0" header when responding to such requests.
 
+Other possible ?p= and ?r= requests were producing appropriate Cache-Control
+headers even before this changeset, and that behavior is left intact:
+
+  https://svn.apache.org/repos/asf/subversion/trunk/README
+  https://svn.apache.org/repos/asf/subversion/trunk/README?p=1716593
+  https://svn.apache.org/repos/asf/subversion/trunk/README?p=1716593&r=1716593
+
 Add a new test that specifies current behavior of how and when we set the
 Cache-Control header, and as well covers the aforementioned ?r= case.