Re: [PATCH] HTTP/2: add logging of RST_STREAM frames with NO_ERROR code

2017-03-29 Thread Piotr Sikora via nginx-devel
Hey Valentin, > Currently such frames are logged with message: > > "client terminated stream %ui with status 0" > > Could you explain why NO_ERROR needs special handling here? The same reason that "client canceled stream %ui" does, it's more appropriate error message. > I haven't found in RFC

Re: [PATCH] HTTP/2: add logging of RST_STREAM frames with NO_ERROR code

2017-03-28 Thread Valentin V. Bartenev
On Sunday 26 March 2017 01:41:20 Piotr Sikora via nginx-devel wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1490516710 25200 > # Sun Mar 26 01:25:10 2017 -0700 > # Node ID 31dfcde3ea2ccf1a2dbd2601ebe8f4306887fc0f > # Parent

[PATCH] HTTP/2: add logging of RST_STREAM frames with NO_ERROR code

2017-03-26 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1490516710 25200 # Sun Mar 26 01:25:10 2017 -0700 # Node ID 31dfcde3ea2ccf1a2dbd2601ebe8f4306887fc0f # Parent 22be63bf21edaa1b8ea916c7d8cd4e5fe4892061 HTTP/2: add logging of RST_STREAM frames with NO_ERROR code.