Welcome! Log In Create A New Profile

Advanced

Re: [PATCH 2 of 2] HTTP/2: fix $body_bytes_sent variable

Valentin V. Bartenev
March 27, 2017 01:06PM
On Sunday 26 March 2017 01:41:10 Piotr Sikora via nginx-devel wrote:
> # HG changeset patch
> # User Piotr Sikora <piotrsikora@google.com>
> # Date 1490516703 25200
> # Sun Mar 26 01:25:03 2017 -0700
> # Node ID dd6c656ed7a327641b2ddfc34768f9551e44bb0f
> # Parent 74ee816e712ee3b731437947470383555653338d
> HTTP/2: fix $body_bytes_sent variable.
>
> Previously, its value included payloads and frame headers of HEADERS
> and CONTINUATION frames.
>
> Signed-off-by: Piotr Sikora <piotrsikora@google.com>
>
> diff -r 74ee816e712e -r dd6c656ed7a3 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
> @@ -1211,6 +1211,9 @@ ngx_http_v2_headers_frame_handler(ngx_ht
> "http2:%ui HEADERS frame %p was sent",
> stream->node->id, frame);
>
> + stream->request->header_size += NGX_HTTP_V2_FRAME_HEADER_SIZE
> + + frame->length;
> +
> ngx_http_v2_handle_frame(stream, frame);
>
> ngx_http_v2_handle_stream(h2c, stream);

Looks good.

wbr, Valentin V. Bartenev

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH 2 of 2] HTTP/2: fix $body_bytes_sent variable

Piotr Sikora via nginx-devel 525 March 26, 2017 04:44AM

Re: [PATCH 2 of 2] HTTP/2: fix $body_bytes_sent variable

Valentin V. Bartenev 301 March 27, 2017 01:06PM

Re: [PATCH 2 of 2] HTTP/2: fix $body_bytes_sent variable

Valentin V. Bartenev 312 March 29, 2017 09:00AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 182
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready