Welcome! Log In Create A New Profile

Advanced

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

Pavel Pautov via nginx-devel
May 12, 2022 06:28PM
Hello,

> -----Original Message-----
> From: Maxim Dounin <mdounin@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)?

_______________________________________________
nginx-devel mailing list -- nginx-devel@nginx.org
To unsubscribe send an email to nginx-devel-leave@nginx.org
Subject Author Views Posted

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

Pavel Pautov via nginx-devel 82 May 12, 2022 01:40AM

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

Maxim Dounin 9 May 12, 2022 06:10PM

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

Pavel Pautov via nginx-devel 12 May 12, 2022 06:28PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 81
Record Number of Users: 6 on February 13, 2018
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready