# HG changeset patch # User Igor Sysoev # Date 1116187200 -14400 # Node ID 0790a85992481e0833e80f4c9fd90dc0985cb9ae # Parent 345d4a254bfb9555b3476d6ee633f8a698857d49 nginx 0.1.31 *) Bugfix: the response encrypted by SSL may not transferred complete. *) Bugfix: errors while processing FastCGI response by SSI. *) Bugfix: errors while using SSI and gzipping. *) Bugfix: the redirect with the 301 code was transferred without response body; bug appeared in 0.1.30. diff --git a/CHANGES b/CHANGES --- a/CHANGES +++ b/CHANGES @@ -1,3 +1,15 @@ + +Changes with nginx 0.1.31 16 May 2005 + + *) Bugfix: the response encrypted by SSL may not transferred complete. + + *) Bugfix: errors while processing FastCGI response by SSI. + + *) Bugfix: errors while using SSI and gzipping. + + *) Bugfix: the redirect with the 301 code was transferred without + response body; bug appeared in 0.1.30. + Changes with nginx 0.1.30 14 May 2005 diff --git a/CHANGES.ru b/CHANGES.ru --- a/CHANGES.ru +++ b/CHANGES.ru @@ -1,3 +1,17 @@ + +Изменения в nginx 0.1.31 16.05.2005 + + *) Исправление: при использовании SSL ответ мог передаваться не до + конца. + + *) Исправление: ошибки при обработке SSI в ответе, полученного от + FastCGI-сервера. + + *) Исправление: ошибки при использовании SSI и сжатия. + + *) Исправление: редирект с кодом 301 передавался без тела ответа; + ошибка появилась в 0.1.30. + Изменения в nginx 0.1.30 14.05.2005 diff --git a/src/core/nginx.h b/src/core/nginx.h --- a/src/core/nginx.h +++ b/src/core/nginx.h @@ -8,7 +8,7 @@ #define _NGINX_H_INCLUDED_ -#define NGINX_VER "nginx/0.1.30" +#define NGINX_VER "nginx/0.1.31" #define NGINX_VAR "NGINX" #define NGX_NEWPID_EXT ".newbin" diff --git a/src/event/ngx_event_openssl.c b/src/event/ngx_event_openssl.c --- a/src/event/ngx_event_openssl.c +++ b/src/event/ngx_event_openssl.c @@ -288,8 +288,9 @@ ngx_ssl_send_chain(ngx_connection_t *c, return NGX_CHAIN_ERROR; } - if (n < 0) { - n = 0; + if (n == NGX_AGAIN) { + c->buffered = 1; + return in; } in->buf->pos += n; diff --git a/src/http/modules/ngx_http_gzip_filter_module.c b/src/http/modules/ngx_http_gzip_filter_module.c --- a/src/http/modules/ngx_http_gzip_filter_module.c +++ b/src/http/modules/ngx_http_gzip_filter_module.c @@ -367,7 +367,7 @@ ngx_http_gzip_header_filter(ngx_http_req r->headers_out.content_length = NULL; } - r->filter_need_in_memory = 1; + r->main_filter_need_in_memory = 1; return ngx_http_next_header_filter(r); } diff --git a/src/http/modules/ngx_http_ssi_filter_module.c b/src/http/modules/ngx_http_ssi_filter_module.c --- a/src/http/modules/ngx_http_ssi_filter_module.c +++ b/src/http/modules/ngx_http_ssi_filter_module.c @@ -321,7 +321,6 @@ ngx_http_ssi_header_filter(ngx_http_requ ctx->timefmt.data = (u_char *) "%A, %d-%b-%Y %H:%M:%S %Z"; r->filter_need_in_memory = 1; - r->filter_ssi_need_in_memory = 1; if (r->main == NULL) { r->headers_out.content_length_n = -1; @@ -464,10 +463,11 @@ ngx_http_ssi_body_filter(ngx_http_reques ngx_memcpy(b, ctx->buf, sizeof(ngx_buf_t)); + b->pos = ctx->copy_start; + b->last = ctx->copy_end; + b->shadow = NULL; b->last_buf = 0; b->recycled = 0; - b->pos = ctx->copy_start; - b->last = ctx->copy_end; if (b->in_file) { if (conf->min_file_chunk < (size_t) (b->last - b->pos)) diff --git a/src/http/ngx_http_copy_filter_module.c b/src/http/ngx_http_copy_filter_module.c --- a/src/http/ngx_http_copy_filter_module.c +++ b/src/http/ngx_http_copy_filter_module.c @@ -88,7 +88,8 @@ ngx_http_copy_filter(ngx_http_request_t ngx_http_set_ctx(r, ctx, ngx_http_copy_filter_module); ctx->sendfile = r->connection->sendfile; - ctx->need_in_memory = r->filter_need_in_memory; + ctx->need_in_memory = r->main_filter_need_in_memory + || r->filter_need_in_memory; ctx->need_in_temp = r->filter_need_temporary; ctx->pool = r->pool; diff --git a/src/http/ngx_http_core_module.c b/src/http/ngx_http_core_module.c --- a/src/http/ngx_http_core_module.c +++ b/src/http/ngx_http_core_module.c @@ -1023,6 +1023,8 @@ ngx_http_subrequest(ngx_http_request_t * sr->internal = 1; + sr->main_filter_need_in_memory = r->main_filter_need_in_memory; + ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0, "http subrequest \"%V\"", uri); diff --git a/src/http/ngx_http_postpone_filter_module.c b/src/http/ngx_http_postpone_filter_module.c --- a/src/http/ngx_http_postpone_filter_module.c +++ b/src/http/ngx_http_postpone_filter_module.c @@ -98,7 +98,7 @@ ngx_http_postpone_filter(ngx_http_reques out = in; } - if (out == NULL && r->out == NULL) { + if (out == NULL && r->out == NULL && !r->connection->buffered) { return NGX_OK; } diff --git a/src/http/ngx_http_request.c b/src/http/ngx_http_request.c --- a/src/http/ngx_http_request.c +++ b/src/http/ngx_http_request.c @@ -384,7 +384,7 @@ void ngx_http_init_request(ngx_event_t * c->ssl->no_rcv_shut = 1; } - r->filter_need_in_memory = 1; + r->main_filter_need_in_memory = 1; } #endif diff --git a/src/http/ngx_http_request.h b/src/http/ngx_http_request.h --- a/src/http/ngx_http_request.h +++ b/src/http/ngx_http_request.h @@ -366,8 +366,8 @@ struct ngx_http_request_s { unsigned closed:1; unsigned done:1; + unsigned main_filter_need_in_memory:1; unsigned filter_need_in_memory:1; - unsigned filter_ssi_need_in_memory:1; unsigned filter_need_temporary:1; unsigned filter_allow_ranges:1; diff --git a/src/http/ngx_http_special_response.c b/src/http/ngx_http_special_response.c --- a/src/http/ngx_http_special_response.c +++ b/src/http/ngx_http_special_response.c @@ -300,7 +300,7 @@ ngx_http_special_response_handler(ngx_ht } else if (error < NGX_HTTP_BAD_REQUEST) { /* 3XX */ - err = error - NGX_HTTP_MOVED_PERMANENTLY; + err = error - NGX_HTTP_MOVED_PERMANENTLY + NGX_HTTP_LEVEL_200; } else if (error < NGX_HTTP_NGX_CODES) { /* 4XX */ diff --git a/src/http/ngx_http_write_filter_module.c b/src/http/ngx_http_write_filter_module.c --- a/src/http/ngx_http_write_filter_module.c +++ b/src/http/ngx_http_write_filter_module.c @@ -168,17 +168,6 @@ ngx_http_write_filter(ngx_http_request_t return NGX_OK; } -#if 0 - /* - * avoid the output if there are no incoming bufs but there are - * the postponed requests or data - */ - - if (in == NULL && r->postponed) { - return NGX_OK; - } -#endif - if (c->write->delayed) { return NGX_AGAIN; } @@ -232,7 +221,7 @@ ngx_http_write_filter(ngx_http_request_t r->out = chain; - if (chain || (last && c->buffered)) { + if (chain || c->buffered) { return NGX_AGAIN; }