Get it, thanks.

On Fri, Jun 9, 2017 at 7:45 PM, Valentin V. Bartenev <vbart@nginx.com>
wrote:

> On Friday 09 June 2017 13:40:23 洪志道 wrote:
> > Hi, Valentin.
> >
> > "
> > Also please note that with your patch clients are still able to
> > negotiate HTTP/2 even if nginx doesn't announce it.
> > "
> >
> > Two points:
> > 1. The patch forbids the clients explicitly not support HTTP/2 doing v2 (
> > ngx_http_v2_init).
> > How to follow you mean of "with the patch, clients are still able to
> > negotiate HTTP/2"
> > 2. "even if nginx doesn't announce it"
> > Is it related to nginx?
> >
> [..]
>
> Your patch prevents advertising the protocol using ALPN and NPN, but
> selecting protocol happens here (this part of code isn't touched):
> http://hg.nginx.org/nginx/file/tip/src/http/ngx_http_request.c#l808
>
> In case of NPN, client can select protocol even if it hasn't been
> advertised by the server.
>
> wbr, Valentin V. Bartenev
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel
>
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel