[PATCH] Upstream keepalive: remove extra SSL shutdown processing.

Pavel Pautov P.Pautov at F5.com
Thu May 12 22:27:10 UTC 2022


Hello,

> -----Original Message-----
> From: Maxim Dounin <mdounin at mdounin.ru>
> ...
> > +++ b/src/http/modules/ngx_http_upstream_keepalive_module.c
> > @@ -471,10 +471,7 @@ ngx_http_upstream_keepalive_close(ngx_co
> >          c->ssl->no_wait_shutdown = 1;
> >          c->ssl->no_send_shutdown = 1;
> >
> > -        if (ngx_ssl_shutdown(c) == NGX_AGAIN) {
> > -            c->ssl->handler = ngx_http_upstream_keepalive_close;
> > -            return;
> > -        }
> > +        (void) ngx_ssl_shutdown(c);
> >      }
> >
> >  #endif
> 
> No, thanks.

So, is it possible for ngx_ssl_shutdown() to return NGX_AGAIN here (and in ngx_http_upstream_next)?



More information about the nginx-devel mailing list