Re: mod_cache: allowing urls ending in "/" to be cached

2004-08-01 Thread Justin Erenkrantz
--On Friday, July 16, 2004 11:29 PM -0400 Bill Stoddard <[EMAIL PROTECTED]> wrote: Pier Fumagalli wrote: I don't understand why mod_cache forcedly avoids caching URLs ending with the "/" (slash) character. Yah, when I went through mod_cache tonight, I tossed that same code, too. I agree that it

Re: mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread Bill Stoddard
Pier Fumagalli wrote: I don't understand why mod_cache forcedly avoids caching URLs ending with the "/" (slash) character. Apparently EGP (who's he?) agrees. Anyhow, it's several months that we're running with this patch in production, and nothing bad seems to be happening. Pier diff -U3 -w

Re: mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread André Malo
* "William A. Rowe, Jr." <[EMAIL PROTECTED]> wrote: > At 09:13 AM 7/16/2004, Graham Leggett wrote: > >Brian Akins wrote: > > > >>Because the cache_in filter gets added in the quick_handler. The > >fast_internal_redirect in mod_dir which translates / to /index.html (or > >whatever) seems to lose t

Re: mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread William A. Rowe, Jr.
At 09:13 AM 7/16/2004, Graham Leggett wrote: >Brian Akins wrote: > >>Because the cache_in filter gets added in the quick_handler. The >>fast_internal_redirect in mod_dir which translates / to /index.html (or whatever) >>seems to lose this filter, so it never gets cached. > >Then I'd say the fix

Re: mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread Matthieu Estrade
Pier Fumagalli wrote: On 16 Jul 2004, at 15:13, Graham Leggett wrote: Brian Akins wrote: If it works, I say commit the patch. Can think of no reason why we should not cache an URL ending is /. Because the cache_in filter gets added in the quick_handler. The fast_internal_redirect in mod_dir wh

Re: mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread Pier Fumagalli
On 16 Jul 2004, at 15:13, Graham Leggett wrote: Brian Akins wrote: If it works, I say commit the patch. Can think of no reason why we should not cache an URL ending is /. Because the cache_in filter gets added in the quick_handler. The fast_internal_redirect in mod_dir which translates / to /in

Re: mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread Graham Leggett
Brian Akins wrote: If it works, I say commit the patch. Can think of no reason why we should not cache an URL ending is /. Because the cache_in filter gets added in the quick_handler. The fast_internal_redirect in mod_dir which translates / to /index.html (or whatever) seems to lose this filte

Re: mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread Brian Akins
Graham Leggett wrote: If it works, I say commit the patch. Can think of no reason why we should not cache an URL ending is /. Because the cache_in filter gets added in the quick_handler. The fast_internal_redirect in mod_dir which translates / to /index.html (or whatever) seems to lose this fil

Re: mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread Joshua Slive
On Fri, 16 Jul 2004, Graham Leggett wrote: Pier Fumagalli wrote: I don't understand why mod_cache forcedly avoids caching URLs ending with the "/" (slash) character. Apparently EGP (who's he?) agrees. Anyhow, it's several months that we're running with this patch in production, and nothing bad s

Re: mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread Graham Leggett
Pier Fumagalli wrote: I don't understand why mod_cache forcedly avoids caching URLs ending with the "/" (slash) character. Apparently EGP (who's he?) agrees. Anyhow, it's several months that we're running with this patch in production, and nothing bad seems to be happening. If it works, I say co

mod_cache: allowing urls ending in "/" to be cached

2004-07-16 Thread Pier Fumagalli
I don't understand why mod_cache forcedly avoids caching URLs ending with the "/" (slash) character. Apparently EGP (who's he?) agrees. Anyhow, it's several months that we're running with this patch in production, and nothing bad seems to be happening. Pier diff -U3 -wr httpd-2.0.50-ori