[PATCH] HTTP/2: add debug logging of control frames
Piotr Sikora
piotrsikora at google.com
Tue May 30 21:21:05 UTC 2017
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.
Best regards,
Piotr Sikora
More information about the nginx-devel
mailing list