cache purge

Johan Bergström johan at bergstroem.nu
Tue Nov 17 10:53:08 MSK 2009


Hey,
while you're patching - please update the example as well. We found (a while ago, sorry for not letting you know) that since location matches "omits" args from $1, we need to add it again. proper example should be:
       location ~ /purge(/.*) {
           allow 127.0.0.1;
           deny all;
           proxy_cache_purge tmpcache $1$is_args$args; 
       }

On 17 nov 2009, at 08.23, Piotr Sikora wrote:

>> One of my developers did the patch manually and she might have introduced some bugs during the process.
> 
> I doubt that she introduced any, because patch for nginx-0.8.8 applies cleanly (without any modifications) against latest nginx.
> 
> Anyway, this "bug" you were seeing appeared when original patch was applied against nginx-0.8.11 or newer (long story short: r->cache->body_start was never set and nginx tried to read 0 bytes). This is fixed now.
> 
>> Would it be easier to make it a standalone module rather than a patch for users?
> 
> It was created as a patch, because we (Johan Bergstr?m from yo.se, who funded this patch and me) hoped that it would be imported to nginx's base someday.
> 
>> I'm waiting for your new patch to the latest nginx and I'll test it out and report back.
> 
> Patch against nginx-0.8.26 is available at:
> http://labs.frickle.com/nginx_cache_purge/
> 
> In the future, please report when you find bugs ;)
> 
> Best regards,
> Piotr Sikora < piotr.sikora at frickle.com >
> 
> 






More information about the nginx mailing list