<div dir="ltr">Thanks.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 9, 2017 at 1:09 AM, Valentin V. Bartenev <span dir="ltr"><<a href="mailto:vbart@nginx.com" target="_blank">vbart@nginx.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Friday 09 June 2017 00:08:06 洪志道 wrote:<br>
> "<br>
><br>
>    For "https" resources, connection reuse additionally depends on<br>
>    having a certificate that is valid for the host in the URI.  The<br>
>    certificate presented by the server MUST satisfy any checks that the<br>
>    client would perform when forming a new TLS connection for the host<br>
>    in the URI.<br>
><br>
> "<br>
><br>
><br>
> It seems the brower can prevent the unreasonable behavior.<br>
><br>
><br>
> In reallity, It still exist some clients that dosen't perfom well in http2.<br>
><br>
> So it's kind of valuable to enable http2 by server.<br>
><br>
><br>
> It's not a good idea the put the patch in nginx,<br>
><br>
> Can you help to check the patch whether contains serious problem?<br>
><br>
><br>
> Maybe it's helpful for other guys.<br>
><br>
><br>
> Thanks again.<br>
><br>
</span>[..]<br>
<br>
The most serious problem with the patch, that it gives an illusion<br>
that HTTP/2 can be enabled per virtual server basis, but in fact it<br>
doesn't prevent requests to any server on particular listen socket<br>
using already existing HTTP/2 connection.<br>
<br>
Also please note that with your patch clients are still able to<br>
negotiate HTTP/2 even if nginx doesn't announce it.<br>
<br>
I don't see any other serious problems.<br>
<div class="HOEnZb"><div class="h5"><br>
  wbr, Valentin V. Bartenev<br>
<br>
______________________________<wbr>_________________<br>
nginx-devel mailing list<br>
<a href="mailto:nginx-devel@nginx.org">nginx-devel@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx-devel" rel="noreferrer" target="_blank">http://mailman.nginx.org/<wbr>mailman/listinfo/nginx-devel</a></div></div></blockquote></div><br></div>