details: http://hg.nginx.org/nginx/rev/afc60bd9008f
branches:
changeset: 6952:afc60bd9008f
user: Piotr Sikora <piotrsikora@google.com>
date: Sun Mar 26 01:25:02 2017 -0700
description:
HTTP/2: fix $bytes_sent variable.

Previously, its value accounted for payloads of HEADERS, CONTINUATION
and DATA frames, as well as frame headers of HEADERS and DATA frames,
but it didn't account for frame headers of CONTINUATION frames.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>

diffstat:

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

diffs (12 lines):

diff -r ce37362a7a70 -r afc60bd9008f src/http/v2/ngx_http_v2_filter_module.c
--- a/src/http/v2/ngx_http_v2_filter_module.c Tue Mar 28 18:15:42 2017 +0300
+++ b/src/http/v2/ngx_http_v2_filter_module.c Sun Mar 26 01:25:02 2017 -0700
@@ -769,6 +769,8 @@ ngx_http_v2_create_headers_frame(ngx_htt
rest -= frame_size;

if (rest) {
+ frame->length += NGX_HTTP_V2_FRAME_HEADER_SIZE;
+
type = NGX_HTTP_V2_CONTINUATION_FRAME;
flags = NGX_HTTP_V2_NO_FLAG;
continue;
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel