Nginx-0.8.31

Changes with nginx 0.8.31 23 Dec
2009

*) Feature: now the "error_page" directive may redirect the 301 and 

302
responses.

*) Feature: the $geoip_city_continent_code, $geoip_latitude, and
   $geoip_longitude variables.
   Thanks to Arvind Sundararajan.

*) Feature: now the ngx_http_image_filter_module deletes always EXIF
   and other application specific data if the data consume more than 

5%
of a JPEG file.

*) Bugfix: nginx closed a connection if a cached response had an 

empty
body.
Thanks to Piotr S…

*) Bugfix: nginx might not be built by gcc 4.x if the -O2 or higher
   optimization option was used.
   Thanks to Maxim D. and Denis F. Latypoff.

*) Bugfix: regular expressions in location were always tested in
   case-sensitive mode; the bug had appeared in 0.8.25.

*) Bugfix: nginx cached a 304 response if there was the 

“If-None-Match”
header line in a proxied request.
Thanks to Tim Dettrick and David K…

*) Bugfix: nginx/Windows tried to delete a temporary file twice if 

the
file should replace an already existent file.


Igor S.
http://sysoev.ru/en/

Hello!

On Wed, Dec 23, 2009 at 06:53:29PM +0300, Igor S. wrote:

[…]

*) Bugfix: nginx closed a connection if a cached response had an empty 
   body.
   Thanks to Piotr S..

I belive setting r->header_only here is wrong aproach. At least
it breaks addition filter module for such responses.

Also you probably want to consider doubled last chunk for stale
responses as fixed here:

http://nginx.org/pipermail/nginx/2009-October/015789.html

Maxim D.

Hello Maxim,

http://nginx.org/pipermail/nginx/2009-October/015789.html

You are right, this will break addition filter module.

Your patch works as expected, I wonder why it wasn’t applied in the
first
place?

Anyway, I modified your patch a little to avoid unnessesary call to
ngx_pcalloc. Hopefully Igor will include this in nginx-0.8.32 :wink:

Best regards,
Piotr S. < [email protected] >