[PATCH 2 of 3] HTTP/2: added support for trailers in HTTP responses
Piotr Sikora
piotrsikora at google.com
Sat Jun 17 20:48:33 UTC 2017
Hey Valentin,
> I've overlooked this while doing previous review, but it looks strange.
>
> Why do you use NGX_LOG_WARN for trailers headers? It results in
> finalizing request with an error (in case of HTTP/2 it means RST_STREAM).
>
> For main headers the NGX_LOG_CRIT level is used. It looks too serious,
> but the WARN level is too low.
Good catch, thanks!
This was left-over from the initial version, which skipped too long
trailers, instead of resetting stream.
Best regards,
Piotr Sikora
More information about the nginx-devel
mailing list