On 27 Mar 2013, at 6:06 PM, Yann Ylavic <ylavic....@gmail.com> wrote:

> Index: modules/cache/mod_cache.h
> ===================================================================
> --- modules/cache/mod_cache.h (revision 1461557)
> +++ modules/cache/mod_cache.h (working copy)
> @@ -152,9 +152,12 @@
> 
> /* Create a new table consisting of those elements from an output
>  * headers table that are allowed to be stored in a cache;
> + * when cc is not NULL, also strip the headers specified by the
> + * Cache-Control private= or no-cache= directives;
>  * ensure there is a content type and capture any errors.
>  */
> -CACHE_DECLARE(apr_table_t *)ap_cache_cacheable_headers_out(request_rec *r);
> +CACHE_DECLARE(apr_table_t *)ap_cache_cacheable_headers_out(request_rec *r,
> +                                                 const cache_control_t *cc);
> 
> /**
>  * Parse the Cache-Control and Pragma headers in one go, marking

Been snowed under and haven't had a chance to look at this in detail, but one 
quick thing - we would definitely want to be able to backport this to v2.4 so 
as to get it into people's hands, and to do that, we cannot change the public 
APIs. We would need to find a way to do this without changing the API.

A further thing is the reparsing on the Cache-Control string, I'd like to see 
if I can find a way to avoid this, but need to dig as how to do that.

Regards,
Graham
--

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to