Re: upstream sent too big header

["Борис Долгов" <boris@xxxxxxxxxxx>]

Тогда предлагаю патч

21 ноября 2008 г. 22:02 пользователь Maxim Dounin <mdounin@xxxxxxxxxx> написал:

Hello!

On Fri, Nov 21, 2008 at 09:24:26PM +0300, Борис Долгов wrote:

> Похоже на баг:
> В документации к fastcgi_buffer_size - По умолчанию размер буфера равен
> размеру одного буфера в директиве

> fastcgi_buffers<http://sysoev.ru/nginx/docs/http/ngx_http_fastcgi_module.html#fastcgi_buffers>,

> однако его можно сделать меньше.
> И исправилось только после увеличения того самого fastcgi_buffer_size.

Да, эта часть документации не соответствуюет действительности.

И у птичек^W^W в proxy то же самое.

Maxim Dounin

-- 
С уважением, Борис Долгов.
icq 77556665
e-mail boris@xxxxxxxxxxx
diff -Naur nginx-0.7.20/src/http/modules/ngx_http_fastcgi_module.c 
nginx-0.7.20-patched/src/http/modules/ngx_http_fastcgi_module.c
--- nginx-0.7.20/src/http/modules/ngx_http_fastcgi_module.c     2008-09-24 
18:02:50.000000000 +0400
+++ nginx-0.7.20-patched/src/http/modules/ngx_http_fastcgi_module.c     
2008-11-21 22:34:19.000000000 +0300
@@ -1720,14 +1720,13 @@
     ngx_conf_merge_size_value(conf->upstream.send_lowat,
                               prev->upstream.send_lowat, 0);
 
-    ngx_conf_merge_size_value(conf->upstream.buffer_size,
-                              prev->upstream.buffer_size,
-                              (size_t) ngx_pagesize);
-
-
     ngx_conf_merge_bufs_value(conf->upstream.bufs, prev->upstream.bufs,
                               8, ngx_pagesize);
 
+    ngx_conf_merge_size_value(conf->upstream.buffer_size,
+                              prev->upstream.buffer_size,
+                              (size_t) conf->upstream.bufs.size);
+
     if (conf->upstream.bufs.num < 2) {
         ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,
                            "there must be at least 2 \"fastcgi_buffers\"");
diff -Naur nginx-0.7.20/src/http/modules/ngx_http_proxy_module.c 
nginx-0.7.20-patched/src/http/modules/ngx_http_proxy_module.c
--- nginx-0.7.20/src/http/modules/ngx_http_proxy_module.c       2008-10-30 
18:59:10.000000000 +0300
+++ nginx-0.7.20-patched/src/http/modules/ngx_http_proxy_module.c       
2008-11-21 22:37:14.000000000 +0300
@@ -1736,12 +1736,13 @@
     ngx_conf_merge_size_value(conf->upstream.send_lowat,
                               prev->upstream.send_lowat, 0);
 
+    ngx_conf_merge_bufs_value(conf->upstream.bufs, prev->upstream.bufs,
+                              8, ngx_pagesize);
+
     ngx_conf_merge_size_value(conf->upstream.buffer_size,
                               prev->upstream.buffer_size,
-                              (size_t) ngx_pagesize);
+                              (size_t) conf->upstream.bufs.size);
 
-    ngx_conf_merge_bufs_value(conf->upstream.bufs, prev->upstream.bufs,
-                              8, ngx_pagesize);
 
     if (conf->upstream.bufs.num < 2) {
         ngx_conf_log_error(NGX_LOG_EMERG, cf, 0,