On Tuesday 26 January 2016 14:54:56 Valentin V. Bartenev wrote:
> On Tuesday 26 January 2016 16:00:08 Shengtuo Hu wrote:
> > Hi,
> >
> > Another error I met recently. It occurred when both padding and
> > continuation are enabled.
> >
> > A normal HEADERS frame was divided as follows:
> > HEADERS ===> HEADERS(PADDED_FLAG) + CONTINUATION + CONTINUATION
> >
> > After sending these frames, I got an error. In the debug log file, I found
> > "client sent inappropriate frame while CONTINUATION was expected while
> > processing HTTP/2 connection". Then I read the source code (v 1.9.9), and
> > located the function "ngx_http_v2_handle_continuation" (ngx_http_v2.c, line
> > 1749). It seems NGINX does not skip the "padding part", but tries to read
> > "type" field in the next CONTINUATION frame directly.
> >
> [..]
>
> Yes, you're right. It isn't able to skip padding between HEADERS and
> CONTINUATION frames.
>

That has been fixed: http://hg.nginx.org/nginx/rev/0e0e2e522fa2

Thanks for the report.

wbr, Valentin V. Bartenev

_______________________________________________
nginx mailing list
nginx@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx