Maxim Dounin
November 30, 2022 08:42PM
Hello!

On Wed, Nov 30, 2022 at 07:29:35PM +0400, Sergey Kandaurov wrote:

> > On 29 Nov 2022, at 02:12, Maxim Dounin <mdounin@mdounin.ru> wrote:
> >
> > Hello!
> >
> > The following patch fixes CPU hog observed with unbuffered SSL proxying
> > after SSL errors (ticket #2418). Fix is to always clear c->read->ready
> > flag when returning errors from ngx_ssl_recv().
> >
> > An additional patch cleans up some win32-specific edge cases (not expected
> > to appear in practice though) when c->read->ready is not cleared when
> > errors or EOFs are returned from ngx_wsarecv() and ngx_wsarecv_chain().
> >
>
> Hello!
>
> Both patches look good to me.

Pushed to http://mdounin.ru/hg/nginx, thanks.

--
Maxim Dounin
http://mdounin.ru/
_______________________________________________
nginx-devel mailing list -- nginx-devel@nginx.org
To unsubscribe send an email to nginx-devel-leave@nginx.org
Subject Author Views Posted

[PATCH 0 of 2] unbuffered proxying CPU hog (ticket #2418)

Maxim Dounin 378 November 28, 2022 05:22PM

[PATCH 2 of 2] Win32: event flags handling edge cases in ngx_wsarecv()

Maxim Dounin 88 November 28, 2022 05:22PM

Re: [PATCH 0 of 2] unbuffered proxying CPU hog (ticket #2418)

Sergey Kandaurov 93 November 30, 2022 10:32AM

Re: [PATCH 0 of 2] unbuffered proxying CPU hog (ticket #2418)

Maxim Dounin 107 November 30, 2022 08:42PM



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

Online Users

Guests: 319
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready