[PATCH 1 of 2] HTTP/2: fix $bytes_sent variable

Piotr Sikora piotrsikora at google.com
Sun Mar 26 08:41:09 UTC 2017


# HG changeset patch
# User Piotr Sikora <piotrsikora at google.com>
# Date 1490516702 25200
#      Sun Mar 26 01:25:02 2017 -0700
# Node ID 74ee816e712ee3b731437947470383555653338d
# Parent  22be63bf21edaa1b8ea916c7d8cd4e5fe4892061
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>

diff -r 22be63bf21ed -r 74ee816e712e 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
@@ -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