diff options
| author | Daniel Silverstone <dsilvers@netsurf-browser.org> | 2010-03-28 16:51:47 +0000 |
|---|---|---|
| committer | Daniel Silverstone <dsilvers@netsurf-browser.org> | 2010-03-28 16:51:47 +0000 |
| commit | 66d141850656222f5e365d11d4824e198e291e12 (patch) | |
| tree | bbcaf571284a69926968d3a36c888d1b77cd33cc /content | |
| parent | 74108d1e09f42c61d48219edc578d04c8df9cd13 (diff) | |
| download | netsurf-66d141850656222f5e365d11d4824e198e291e12.tar.gz netsurf-66d141850656222f5e365d11d4824e198e291e12.tar.bz2 | |
Allow us to cache foo?bar when it explicitly allows it. as per rfc2616 13.9
svn path=/trunk/netsurf/; revision=10190
Diffstat (limited to 'content')
| -rw-r--r-- | content/llcache.c | 23 |
1 files changed, 15 insertions, 8 deletions
diff --git a/content/llcache.c b/content/llcache.c index 0c7c2b83c..3dd39916a 100644 --- a/content/llcache.c +++ b/content/llcache.c @@ -105,7 +105,8 @@ struct llcache_object { llcache_object *next; /**< Next in list */ char *url; /**< Post-redirect URL for object */ - + bool has_query; /**< URL has a query segment */ + /** \todo We need a generic dynamic buffer object */ uint8_t *source_data; /**< Source data for object */ size_t source_len; /**< Byte length of source data */ @@ -464,10 +465,7 @@ nserror llcache_object_retrieve(const char *url, uint32_t flags, * Caching Rules: * * 1) Forced fetches are never cached - * 2) GET requests with query segments are never cached - * 3) POST requests are never cached - * - * \todo Find out if restriction (2) can be removed + * 2) POST requests are never cached */ /* Look for a query segment */ @@ -479,7 +477,7 @@ nserror llcache_object_retrieve(const char *url, uint32_t flags, url_destroy_components(&components); - if (flags & LLCACHE_RETRIEVE_FORCE_FETCH || has_query || post != NULL) { + if (flags & LLCACHE_RETRIEVE_FORCE_FETCH || post != NULL) { /* Create new object */ error = llcache_object_new(url, &obj); if (error != NSERROR_OK) @@ -502,7 +500,9 @@ nserror llcache_object_retrieve(const char *url, uint32_t flags, /* Returned object is already in the cached list */ } - + + obj->has_query = has_query; + *result = obj; return NSERROR_OK; @@ -1272,7 +1272,14 @@ void llcache_fetch_callback(fetch_msg msg, void *p, const void *data, case FETCH_DATA: /* Received some data */ object->fetch.state = LLCACHE_FETCH_DATA; - + if (object->has_query && + (object->cache.expires == 0 && object->cache.max_age == INVALID_AGE)) { + /* URI had query string and did not provide an explicit expiration + * time, thus by rfc2616 13.9 we must invalidate the cache data + * to force the cache to not retain the object. + */ + memset(&(object->cache), 0, sizeof(llcache_cache_control)); + } error = llcache_fetch_process_data(object, data, size); break; case FETCH_FINISHED: |