Re: [PATCH] Dynamic rate limiting for limit_req module

2023-01-06 Thread J Carter
Hello, Please find below my revised patch with style and code bug fixes included. The only change in behavior of note is that invalid rate values are now simply ignored in the same fashion an empty key is, and the use of the parsing of complex value being disabled if a non variable is used

[njs] Fixed Object.values() and Object.entries() with external objects.

2023-01-06 Thread Dmitry Volyntsev
details: https://hg.nginx.org/njs/rev/a98b63e87688 branches: changeset: 2024:a98b63e87688 user: Dmitry Volyntsev date: Fri Jan 06 16:50:46 2023 -0800 description: Fixed Object.values() and Object.entries() with external objects. This closes #606 issue on Github. diffstat:

[njs] Fixed unitialized value usage warning.

2023-01-06 Thread Dmitry Volyntsev
details: https://hg.nginx.org/njs/rev/1a5d22feadc2 branches: changeset: 2023:1a5d22feadc2 user: Dmitry Volyntsev date: Thu Jan 05 22:05:03 2023 -0800 description: Fixed unitialized value usage warning. Found by Coverity (CID 1518908). diffstat: external/njs_webcrypto_module.c |

Re: [PATCH 1 of 3] QUIC: set stream error flag on reset

2023-01-06 Thread Sergey Kandaurov
> On 5 Jan 2023, at 19:31, Roman Arutyunyan wrote: > > src/event/quic/ngx_event_quic_streams.c | 14 -- > 1 files changed, 12 insertions(+), 2 deletions(-) > > > # HG changeset patch > # User Roman Arutyunyan > # Date 1672932528 -14400 > # Thu Jan 05 19:28:48 2023 +0400 > #

Re: [PATCH 2 of 6] QUIC: handle datagrams directly in ngx_quic_recvmsg()

2023-01-06 Thread Roman Arutyunyan
Hi, Following a discussion with Sergey, here's a new version with a simplified ngx_quic_run(). On Fri, Dec 09, 2022 at 09:38:48AM +, Roman Arutyunyan wrote: > # HG changeset patch > # User Roman Arutyunyan > # Date 1670428292 0 > # Wed Dec 07 15:51:32 2022 + > # Branch quic > #