[PATCH] Upstream: handling of certificates specified as an empty string

Pavel Pautov P.Pautov at F5.com
Tue May 31 21:24:45 UTC 2022


LGTM.

> -----Original Message-----
> From: Sergey Kandaurov <pluknet at nginx.com>
> Sent: Monday, May 30, 2022 15:54
> To: nginx-devel at nginx.org
> Cc: Pavel Pautov <P.Pautov at F5.com>
> Subject: Re: [PATCH] Upstream: handling of certificates specified as an empty
> string
> 
> EXTERNAL MAIL: pluknet at nginx.com
> 
> > On 27 May 2022, at 08:26, Pavel Pautov via nginx-devel <nginx-
> devel at nginx.org> wrote:
> >
> > Hi,
> >
> > Shall we restore original behavior for stream proxy as well?
> >
> 
> Fixed, tnx.  The relevant part:
> 
> diff --git a/src/stream/ngx_stream_proxy_module.c
> b/src/stream/ngx_stream_proxy_module.c
> --- a/src/stream/ngx_stream_proxy_module.c
> +++ b/src/stream/ngx_stream_proxy_module.c
> @@ -2244,7 +2244,7 @@ ngx_stream_proxy_set_ssl(ngx_conf_t *cf,
>                  return NGX_ERROR;
>              }
> 
> -        } else {
> +        } else if (pscf->ssl_certificate->value.len) {
>              if (ngx_ssl_certificate(cf, pscf->ssl,
>                                      &pscf->ssl_certificate->value,
>                                      &pscf->ssl_certificate_key->value,
> 
> --
> Sergey Kandaurov
> 



More information about the nginx-devel mailing list