[PATCH] Proxy: add "proxy_ssl_alpn" directive

Maxim Dounin mdounin at mdounin.ru
Thu Jun 8 16:32:08 UTC 2017


Hello!

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

> # HG changeset patch
> # User Piotr Sikora <piotrsikora at 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 at 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/


More information about the nginx-devel mailing list