# HG changeset patch
# User Maxim Dounin <mdounin@mdounin.ru>
# Date 1315040808 -14400
# Node ID 9b75a0e6fc95d5fbd827e4589826e9be488eefc4
# Parent 0e57952574064b354ee3084b6c8b3cac80941c33
Protocol version parsing in ngx_http_parse_status_line().

Once we know protocol version, set u->headers_in.connection_close to indicate
implicitly assumed connection close with HTTP before 1.1.

diff --git a/src/http/modules/ngx_http_proxy_module.c b/src/http/modules/ngx_http_proxy_module.c
--- a/src/http/modules/ngx_http_proxy_module.c
+++ b/src/http/modules/ngx_http_proxy_module.c
@@ -1210,6 +1210,7 @@ ngx_http_proxy_process_status_line(ngx_h

r->http_version = NGX_HTTP_VERSION_9;
u->state->status = NGX_HTTP_OK;
+ u->headers_in.connection_close = 1;

return NGX_OK;
}
@@ -1234,6 +1235,10 @@ ngx_http_proxy_process_status_line(ngx_h
"http proxy status %ui \"%V\"",
u->headers_in.status_n, &u->headers_in.status_line);

+ if (ctx->status.http_version < NGX_HTTP_VERSION_11) {
+ u->headers_in.connection_close = 1;
+ }
+
u->process_header = ngx_http_proxy_process_header;

return ngx_http_proxy_process_header(r);
diff --git a/src/http/ngx_http.h b/src/http/ngx_http.h
--- a/src/http/ngx_http.h
+++ b/src/http/ngx_http.h
@@ -52,6 +52,7 @@ struct ngx_http_log_ctx_s {


typedef struct {
+ ngx_uint_t http_version;
ngx_uint_t code;
ngx_uint_t count;
u_char *start;
diff --git a/src/http/ngx_http_parse.c b/src/http/ngx_http_parse.c
--- a/src/http/ngx_http_parse.c
+++ b/src/http/ngx_http_parse.c
@@ -1403,6 +1403,7 @@ ngx_http_parse_status_line(ngx_http_requ
return NGX_ERROR;
}

+ r->http_major = ch - '0';
state = sw_major_digit;
break;

@@ -1417,6 +1418,7 @@ ngx_http_parse_status_line(ngx_http_requ
return NGX_ERROR;
}

+ r->http_major = r->http_major * 10 + ch - '0';
break;

/* the first digit of minor HTTP version */
@@ -1425,6 +1427,7 @@ ngx_http_parse_status_line(ngx_http_requ
return NGX_ERROR;
}

+ r->http_minor = ch - '0';
state = sw_minor_digit;
break;

@@ -1439,6 +1442,7 @@ ngx_http_parse_status_line(ngx_http_requ
return NGX_ERROR;
}

+ r->http_minor = r->http_minor * 10 + ch - '0';
break;

/* HTTP status code */
@@ -1516,6 +1520,7 @@ done:
status->end = p;
}

+ status->http_version = r->http_major * 1000 + r->http_minor;
r->state = sw_start;

return NGX_OK;

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