Hello,
Didn't see a way to get into the thread after joining the list, but
hopefully this still replies correctly.
First of all thank you for the prompt response and updated patch.
We implemented the updated async open patch, and started testing it in our
lab.
The static module part of the patch
details: https://hg.nginx.org/njs/rev/3a7ffe641a77
branches:
changeset: 1700:3a7ffe641a77
user: Alexander Borisov
date: Thu Sep 02 19:32:27 2021 +0300
description:
Fixed async ctx erasing when a function is called multiple times.
The bug was introduced in 92d10cd761e2.
diffstat:
details: https://hg.nginx.org/njs/rev/cd87eaf03e4f
branches:
changeset: 1701:cd87eaf03e4f
user: Alexander Borisov
date: Thu Sep 02 19:32:34 2021 +0300
description:
Parser: fixed flag setting when parsing function arguments.
The bug was introduced in 92d10cd761e2.
diffstat:
src/nj
details: https://hg.nginx.org/njs/rev/f2410c8b80d9
branches:
changeset: 1702:f2410c8b80d9
user: Alexander Borisov
date: Thu Sep 02 19:32:52 2021 +0300
description:
Parser: removed dead store in await;
The bug was introduced in 92d10cd761e2.
diffstat:
src/njs_parser.c | 2 --
1 f
details: https://hg.nginx.org/njs/rev/502be9e80244
branches:
changeset: 1703:502be9e80244
user: Alexander Borisov
date: Thu Sep 02 19:33:03 2021 +0300
description:
Parser: catching errors parsing in await expression.
The bug was introduced in 92d10cd761e2.
diffstat:
src/njs_parse
details: https://hg.nginx.org/njs/rev/68f8f7ead0fc
branches:
changeset: 1704:68f8f7ead0fc
user: Alexander Borisov
date: Thu Sep 02 19:33:32 2021 +0300
description:
Added async support for object initializer.
diffstat:
src/njs_async.c | 14 +++---
src/njs_generator.c
Hi, All.
In the function ngx_http_init_phase_handlers, I found two places hard
to understand.
a) The n variable is as known as the next phase handler index. In the
switch case NGX_HTTP_ACCESS_PHASE, it invokes n++, which seems to be
redundant and wrong, While in switch case NGX_HTTP_POST_ACCESS_P
