[nginx] HTTP/2: fix $bytes_sent variable.

Valentin Bartenev vbart at nginx.com
Wed Mar 29 12:54:29 UTC 2017


details:   http://hg.nginx.org/nginx/rev/afc60bd9008f
branches:  
changeset: 6952:afc60bd9008f
user:      Piotr Sikora <piotrsikora at 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 at 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;


More information about the nginx-devel mailing list