Hello!

On Sat, Jun 03, 2017 at 08:04:02PM -0700, Piotr Sikora via nginx-devel wrote:

> # HG changeset patch
> # User Piotr Sikora <piotrsikora@google.com>
> # Date 1489621682 25200
> # Wed Mar 15 16:48:02 2017 -0700
> # Node ID 7733d946e2651a2486a53d912703e2dfaea30421
> # Parent 716852cce9136d977b81a2d1b8b6f9fbca0dce49
> Proxy: add "proxy_ssl_alpn" directive.
>
> ALPN is used here only to indicate which version of the HTTP protocol
> is going to be used and we doesn't verify that upstream agreed to it.
>
> Please note that upstream is allowed to reject SSL connection with a
> fatal "no_application_protocol" alert if it doesn't support it.
>
> Signed-off-by: Piotr Sikora <piotrsikora@google.com>

It doesn't look like this patch make sense by its own.

If this patch is a part of a larger work, please consider
submitting a patch series with the whole feature instead.
Individual patches which doesn't make much sense
by its own are hard to review, and are likely to be rejected.

--
Maxim Dounin
http://nginx.org/
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel