Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] HTTP/2: add debug logging of control frames

Valentin V. Bartenev
June 08, 2017 02:32PM
On Tuesday 30 May 2017 14:21:05 Piotr Sikora via nginx-devel wrote:
> Hey Valentin,
>
> > What do you suggest instead? All 3 params in the same line?
> >
> > http2 send SETTINGS frame MAX_CONCURRENT_STREAMS:%ui
> > INITIAL_WINDOW_SIZE:%uz MAX_FRAME_SIZE:%ud
> >
> > What about receiving part, then? Do you want to put all 6 params in
> > the same line?
> >
> > http2 recv SETTINGS frame HEADER_TABLE_SIZE:%ui (ignored)
> > ENABLE_PUSH:%ui (ignored) MAX_CONCURRENT_STREAMS:%ui (ignored)
> > INITIAL_WINDOW_SIZE:%ui MAX_FRAME_SIZE:%ui MAX_HEADER_LIST_SIZE:%ui
> > (ignored)
> >
> > It makes this way less readable, IMHO.
>
> Ping.
>

Ok, I've already resigned myself to multiline output, but don't let it
look like an another SETTINGS frame.

IMHO, something like that will be good enough:

http2 send SETTINGS frame
http2 SETTINGS param MAX_CONCURRENT_STREAMS: 100
http2 SETTINGS param INITIAL_WINDOW_SIZE: 65536
http2 SETTINGS param MAX_FRAME_SIZE: 16777215


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] HTTP/2: add debug logging of control frames

Piotr Sikora via nginx-devel 590 April 03, 2017 11:14PM

Re: [PATCH] HTTP/2: add debug logging of control frames

Valentin V. Bartenev 305 April 06, 2017 10:32AM

Re: [PATCH] HTTP/2: add debug logging of control frames

Piotr Sikora via nginx-devel 252 April 07, 2017 05:44PM

Re: [PATCH] HTTP/2: add debug logging of control frames

Valentin V. Bartenev 283 April 17, 2017 02:58PM

Re: [PATCH] HTTP/2: add debug logging of control frames

Piotr Sikora via nginx-devel 284 April 24, 2017 07:04PM

Re: [PATCH] HTTP/2: add debug logging of control frames

Piotr Sikora via nginx-devel 328 May 30, 2017 05:22PM

Re: [PATCH] HTTP/2: add debug logging of control frames

Valentin V. Bartenev 213 June 08, 2017 02:32PM

Re: [PATCH] HTTP/2: add debug logging of control frames

Piotr Sikora via nginx-devel 240 June 19, 2017 10:30AM

[PATCH] HTTP/2: add debug logging of control frames

Piotr Sikora via nginx-devel 240 June 19, 2017 10:30AM

[PATCH] HTTP/2: add debug logging of control frames

Piotr Sikora via nginx-devel 403 April 07, 2017 06:10PM



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

Online Users

Guests: 144
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