You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by je...@apache.org on 2007/05/17 07:43:20 UTC
svn commit: r538807 - in /httpd/httpd/trunk: CHANGES
modules/cache/mod_cache.c
Author: jerenkrantz
Date: Wed May 16 22:43:18 2007
New Revision: 538807
URL: http://svn.apache.org/viewvc?view=rev&rev=538807
Log:
RFC 2616 13.2.1 calls out max-age as being a valid 'explicit expiration time',
so per Sec. 13.9 permit queries with just max-age instead of only Expires.
* modules/cache/mod_cache.c
(cache_save_filter): Be closer to RFC 2616's intent for query arguments.
* CHANGES: Update.
Modified:
httpd/httpd/trunk/CHANGES
httpd/httpd/trunk/modules/cache/mod_cache.c
Modified: httpd/httpd/trunk/CHANGES
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/CHANGES?view=diff&rev=538807&r1=538806&r2=538807
==============================================================================
--- httpd/httpd/trunk/CHANGES [utf-8] (original)
+++ httpd/httpd/trunk/CHANGES [utf-8] Wed May 16 22:43:18 2007
@@ -2,6 +2,9 @@
Changes with Apache 2.3.0
[Remove entries to the current 2.0 and 2.2 section below, when backported]
+ *) mod_cache: Allow caching of requests with query arguments when
+ Cache-Control max-age is explicitly specified. [Justin Erenkrantz]
+
*) mod_so: Solve dev's confusion by reporting expected/seen module
magic signatures when failing with a 'garbled' message, and solve
user's confusion by pointing out 'perhaps compiled for a different
Modified: httpd/httpd/trunk/modules/cache/mod_cache.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/cache/mod_cache.c?view=diff&rev=538807&r1=538806&r2=538807
==============================================================================
--- httpd/httpd/trunk/modules/cache/mod_cache.c (original)
+++ httpd/httpd/trunk/modules/cache/mod_cache.c Wed May 16 22:43:18 2007
@@ -452,11 +452,12 @@
/* if a Expires header is in the past, don't cache it */
reason = "Expires header already expired, not cacheable";
}
- else if (!conf->ignorequerystring && r->parsed_uri.query && exps == NULL) {
- /* if query string present but no expiration time, don't cache it
- * (RFC 2616/13.9)
+ else if (!conf->ignorequerystring && r->parsed_uri.query && exps == NULL &&
+ !ap_cache_liststr(NULL, cc_out, "max-age", NULL)) {
+ /* if a query string is present but no explicit expiration time,
+ * don't cache it (RFC 2616/13.9 & 13.2.1)
*/
- reason = "Query string present but no expires header";
+ reason = "Query string present but no explicit expiration time";
}
else if (r->status == HTTP_NOT_MODIFIED &&
!cache->handle && !cache->stale_handle) {