details: http://hg.nginx.org/nginx/rev/9d61ecbe979e
branches: stable-1.10
changeset: 6760:9d61ecbe979e
user: Valentin Bartenev <vbart@nginx.com>
date: Tue Jul 19 20:22:44 2016 +0300
description:
HTTP/2: prevented output of the HEADERS frame for canceled streams.

It's useless to generate HEADERS if the stream has been canceled already.

diffstat:

src/http/v2/ngx_http_v2_filter_module.c | 8 ++++++--
1 files changed, 6 insertions(+), 2 deletions(-)

diffs (25 lines):

diff --git a/src/http/v2/ngx_http_v2_filter_module.c b/src/http/v2/ngx_http_v2_filter_module.c
--- a/src/http/v2/ngx_http_v2_filter_module.c
+++ b/src/http/v2/ngx_http_v2_filter_module.c
@@ -169,6 +169,12 @@ ngx_http_v2_header_filter(ngx_http_reque
return NGX_OK;
}

+ fc = r->connection;
+
+ if (fc->error) {
+ return NGX_ERROR;
+ }
+
if (r->method == NGX_HTTP_HEAD) {
r->header_only = 1;
}
@@ -259,8 +265,6 @@ ngx_http_v2_header_filter(ngx_http_reque
len += 1 + ngx_http_v2_literal_size("Wed, 31 Dec 1986 18:00:00 GMT");
}

- fc = r->connection;
-
if (r->headers_out.location && r->headers_out.location->value.len) {

if (r->headers_out.location->value.data[0] == '/') {

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel