Re: [PATCH] mod_cache. Allow override of some vary headers

2005-10-28 Thread Brian Akins
Another one. Can we get a vote? Sorry to be a pest, just don't want these to be forgotten. Brian Akins wrote: Here's a new patch that changes the option name to CacheVaryOverride and does some of the stuff Justin recommened.

Re: [PATCH] mod_cache. Allow override of some vary headers

2005-08-18 Thread Brian Akins
[EMAIL PROTECTED] wrote: What sort of testing did you do in those few hours? That's why I posted it, to get some more input. If I could play devil's advocate for a moment... my concern would be that you haven't considered certain scenarios that won't work with this patch. Most of the

Re: [PATCH] mod_cache. Allow override of some vary headers

2005-08-18 Thread Brian Akins
Here's a new patch that changes the option name to CacheVaryOverride and does some of the stuff Justin recommened. -- Brian Akins Lead Systems Engineer CNN Internet Technologies diff -ru httpd-trunk.orig/modules/cache/cache_storage.c httpd-trunk.new/modules/cache/cache_storage.c ---

[PATCH] mod_cache. Allow override of some vary headers

2005-08-17 Thread Akins, Brian
This patch allows one to override the values of some headers so that they vary to the same value. Config Example: #all lines that have gzip set one variable SetEnvIf Accept-Encoding gzip gzip=1 #browsers that have problems with gzip BrowserMatch MSIE [1-3] gzip=0 BrowserMatch MSIE [1-5].*Mac

Re: [PATCH] mod_cache. Allow override of some vary headers

2005-08-17 Thread Joshua Slive
Akins, Brian wrote: This patch allows one to override the values of some headers so that they vary to the same value. CacheOverrideHeader Accept-Encoding gzip CacheOverrideHeader User-Agent gzip Very useful looking. I suggest CacheVaryOverride as a much more clear and precise name.

Re: [PATCH] mod_cache. Allow override of some vary headers

2005-08-17 Thread Justin Erenkrantz
--On August 17, 2005 3:01:03 PM -0400 Akins, Brian [EMAIL PROTECTED] wrote: This patch allows one to override the values of some headers so that they vary to the same value. Config Example: # all lines that have gzip set one variable SetEnvIf Accept-Encoding gzip gzip=1 # browsers that have

Re: [PATCH] mod_cache. Allow override of some vary headers

2005-08-17 Thread Brian Akins
Justin Erenkrantz wrote: The concept is fine. (And as Joshua pointed out 'CacheVaryOverride' would be a better name.) I'm not attatched to the name. So that sounds good to me. A few issues with the implementation (modulo style nits)... This was all done in a few hours today, including

Re: [PATCH] mod_cache. Allow override of some vary headers

2005-08-17 Thread TOKILEY
In a message dated 8/17/2005 2:01:41 PM Central Daylight Time, [EMAIL PROTECTED] writes: CacheOverrideHeader Accept-Encoding gzip CacheOverrideHeader User-Agent gzip This would allow all browsers that send "Accept-Encoding: gzip" and do not match the BrowserMatches to be mapped to the same