summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBill Stoddard <stoddard@apache.org>2004-09-24 13:15:24 +0000
committerBill Stoddard <stoddard@apache.org>2004-09-24 13:15:24 +0000
commita6963e4c7ea78d321e557707ed62008177a57cda (patch)
tree04151ffdb12e7a3304ccbe9458675ac4395b56a3
parent01e1a2e14a2492f8dd9be87e1a9676c8058b6838 (diff)
downloadhttpd-a6963e4c7ea78d321e557707ed62008177a57cda.tar.gz
Fix up some comments in CACHE_SAVE filter and fix a minor style nit.
No functional changes. git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/APACHE_2_0_BRANCH@105286 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r--modules/experimental/mod_cache.c15
1 files changed, 10 insertions, 5 deletions
diff --git a/modules/experimental/mod_cache.c b/modules/experimental/mod_cache.c
index 94a5adb06a..a47424857e 100644
--- a/modules/experimental/mod_cache.c
+++ b/modules/experimental/mod_cache.c
@@ -236,12 +236,15 @@ static int cache_out_filter(ap_filter_t *f, apr_bucket_brigade *bb)
* ---------------
*
* Decide whether or not this content should be cached.
- * If we decide no it should:
+ * If we decide no it should not:
* remove the filter from the chain
* If we decide yes it should:
- * pass the data to the storage manager
- * pass the data to the next filter (the network)
- *
+ * Have we already started saving the response?
+ * If we have started, pass the data to the storage manager via store_body
+ * Otherwise:
+ * Check to see if we *can* save this particular response.
+ * If we can, call cache_create_entity() and save the headers and body
+ * Finally, pass the data to the next filter (the network or whatever)
*/
static int cache_save_filter(ap_filter_t *f, apr_bucket_brigade *in)
@@ -447,6 +450,8 @@ static int cache_save_filter(ap_filter_t *f, apr_bucket_brigade *in)
/* ship the data up the stack */
return ap_pass_brigade(f->next, in);
}
+
+ /* Make it so that we don't execute this path again. */
cache->in_checked = 1;
/* Set the content length if known.
@@ -636,7 +641,7 @@ static int cache_save_filter(ap_filter_t *f, apr_bucket_brigade *in)
info->content_type = apr_pstrdup(r->pool, r->content_type);
info->etag = apr_pstrdup(r->pool, etag);
info->lastmods = apr_pstrdup(r->pool, lastmods);
- info->filename = apr_pstrdup(r->pool, r->filename );
+ info->filename = apr_pstrdup(r->pool, r->filename);
/*
* Write away header information to cache.