[PATCH] HTTP/2: add debug logging of pseudo-headers

Valentin V. Bartenev vbart at nginx.com
Thu Apr 6 14:02:19 UTC 2017


On Monday 03 April 2017 20:13:34 Piotr Sikora via nginx-devel wrote:
> # HG changeset patch
> # User Piotr Sikora <piotrsikora at google.com>
> # Date 1490516711 25200
> #      Sun Mar 26 01:25:11 2017 -0700
> # Node ID 3d72ae17c41990774721a678c50b8307ecb684c8
> # Parent  22be63bf21edaa1b8ea916c7d8cd4e5fe4892061
> HTTP/2: add debug logging of pseudo-headers.
> 
> Signed-off-by: Piotr Sikora <piotrsikora at google.com>
> 
> diff -r 22be63bf21ed -r 3d72ae17c419 src/http/v2/ngx_http_v2.c
> --- a/src/http/v2/ngx_http_v2.c
> +++ b/src/http/v2/ngx_http_v2.c
> @@ -1585,6 +1585,10 @@ ngx_http_v2_state_process_header(ngx_htt
>          rc = ngx_http_v2_pseudo_header(r, header);
>  
>          if (rc == NGX_OK) {
> +            ngx_log_debug2(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
> +                           "http2 http header: \":%V: %V\"",
> +                           &header->name, &header->value);
> +
>              return ngx_http_v2_state_header_complete(h2c, pos, end);
>          }
>  

Maybe "http2 pseudo-header: \":%V: %V\""?
Because it doesn't look like a valid "http header".

  wbr, Valentin V. Bartenev



More information about the nginx-devel mailing list