LGTM. > -----Original Message----- > From: Sergey Kandaurov <[email protected]> > Sent: Monday, May 30, 2022 15:54 > To: [email protected] > Cc: Pavel Pautov <[email protected]> > Subject: Re: [PATCH] Upstream: handling of certificates specified as an empty > string > > EXTERNAL MAIL: [email protected] > > > On 27 May 2022, at 08:26, Pavel Pautov via nginx-devel <nginx- > [email protected]> 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 >
_______________________________________________ nginx-devel mailing list -- [email protected] To unsubscribe send an email to [email protected]
