Re: [PATCH] HTTP/2: add debug logging of pseudo-headers and control frames

2017-04-03 Thread Piotr Sikora via nginx-devel
Hey, > # HG changeset patch > # User Piotr Sikora > # Date 1490516711 25200 > # Sun Mar 26 01:25:11 2017 -0700 > # Node ID 6990fb6463ce47705e06ff6d0fbd9ae6696aeb37 > # Parent 22be63bf21edaa1b8ea916c7d8cd4e5fe4892061 > HTTP/2: add debug logging of pseudo-headers and control frames. > > Signe

[PATCH] HTTP/2: add debug logging of pseudo-headers

2017-04-03 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1490516711 25200 # Sun Mar 26 01:25:11 2017 -0700 # Node ID 3d72ae17c41990774721a678c50b8307ecb684c8 # Parent 22be63bf21edaa1b8ea916c7d8cd4e5fe4892061 HTTP/2: add debug logging of pseudo-headers. Signed-off-by: Piotr Sikora diff -r 22be63bf2

[PATCH] HTTP/2: add debug logging of control frames

2017-04-03 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1490516711 25200 # Sun Mar 26 01:25:11 2017 -0700 # Node ID 06d6418afe6e73604aea707ef9c5802f5bf27bf4 # Parent 22be63bf21edaa1b8ea916c7d8cd4e5fe4892061 HTTP/2: add debug logging of control frames. Signed-off-by: Piotr Sikora diff -r 22be63bf2

Re: [PATCH 2 of 2] Upstream: allow recovery from "429 Too Many Requests" response

2017-04-03 Thread Maxim Dounin
Hello! On Fri, Mar 24, 2017 at 03:47:41AM -0700, Piotr Sikora via nginx-devel wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1490348883 25200 > # Fri Mar 24 02:48:03 2017 -0700 > # Node ID b377cfedf632b14a3d459e12342a0557a25a790c > # Parent 799ef976b58cadbc212bd790a666033d377

Re: [PATCH 1 of 2] HTTP: add support for "429 Too Many Requests" response (RFC6585)

2017-04-03 Thread Maxim Dounin
Hello! On Fri, Mar 24, 2017 at 03:47:40AM -0700, Piotr Sikora via nginx-devel wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1490348883 25200 > # Fri Mar 24 02:48:03 2017 -0700 > # Node ID 799ef976b58cadbc212bd790a666033d3777c10d > # Parent 39ff6939266e913e8bfd400e60f9520e707

[nginx] Upstream: allow recovery from "429 Too Many Requests" response.

2017-04-03 Thread Maxim Dounin
details: http://hg.nginx.org/nginx/rev/fa56ab75cffc branches: changeset: 6966:fa56ab75cffc user: Piotr Sikora date: Fri Mar 24 02:48:03 2017 -0700 description: Upstream: allow recovery from "429 Too Many Requests" response. This change adds "http_429" parameter to "proxy_next_upstre

[nginx] Added support for "429 Too Many Requests" response (RFC6585).

2017-04-03 Thread Maxim Dounin
details: http://hg.nginx.org/nginx/rev/3ef4cadfad7f branches: changeset: 6965:3ef4cadfad7f user: Piotr Sikora date: Fri Mar 24 02:48:03 2017 -0700 description: Added support for "429 Too Many Requests" response (RFC6585). This change adds reason phrase in status line and pretty resp

Re: [PATCH][bugfix] Upstream: clear the delayed flag to prevent blocking from sending.

2017-04-03 Thread Maxim Dounin
Hello! On Mon, Apr 03, 2017 at 04:02:07PM +0800, 胡聪 (hucc) wrote: [...] > At the same time, I noticed that the type of rc is int not ngx_int_t in > ngx_http_writer(). The type should be ngx_int_t, right? There was a time the > return type of ngx_http_writer() is int. Later, it became void. Since

[nginx] Fixed type.

2017-04-03 Thread Maxim Dounin
details: http://hg.nginx.org/nginx/rev/5d3d9b52327d branches: changeset: 6964:5d3d9b52327d user: hucongcong date: Mon Apr 03 14:29:40 2017 +0800 description: Fixed type. diffstat: src/http/ngx_http_request.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diffs (21 li

[njs] Variables may be accessed incorrectly by nested functions.

2017-04-03 Thread Igor Sysoev
details: http://hg.nginx.org/njs/rev/c46da90ca064 branches: changeset: 329:c46da90ca064 user: Igor Sysoev date: Sun Apr 02 12:36:05 2017 +0300 description: Variables may be accessed incorrectly by nested functions. diffstat: njs/njs_parser.c | 5 + njs/test/njs_unit_t

[njs] Large indexes processing has been fixed in

2017-04-03 Thread Igor Sysoev
details: http://hg.nginx.org/njs/rev/8f59eeb8ee2d branches: changeset: 327:8f59eeb8ee2d user: Igor Sysoev date: Sat Apr 01 15:32:04 2017 +0300 description: Large indexes processing has been fixed in Array.prototype.reduceRight(). diffstat: njs/njs_array.c | 2 +- 1 files changed,

[njs] Array iterators optimizations.

2017-04-03 Thread Igor Sysoev
details: http://hg.nginx.org/njs/rev/cee288760080 branches: changeset: 328:cee288760080 user: Igor Sysoev date: Sun Apr 02 12:35:11 2017 +0300 description: Array iterators optimizations. diffstat: njs/njs_array.c | 221 +- njs/

[nginx] Slice filter: allowed at most one subrequest at a time.

2017-04-03 Thread Roman Arutyunyan
details: http://hg.nginx.org/nginx/rev/a97ad1663ef4 branches: changeset: 6962:a97ad1663ef4 user: Roman Arutyunyan date: Tue Mar 28 14:03:57 2017 +0300 description: Slice filter: allowed at most one subrequest at a time. Previously, if slice main request write handler was called whil

[nginx] Slice filter: prevented slice redirection (ticket #1219).

2017-04-03 Thread Roman Arutyunyan
details: http://hg.nginx.org/nginx/rev/3ff293cfdab8 branches: changeset: 6963:3ff293cfdab8 user: Roman Arutyunyan date: Fri Mar 31 21:47:56 2017 +0300 description: Slice filter: prevented slice redirection (ticket #1219). When a slice subrequest was redirected to a new location, its

Re: [PATCH 3 of 3] Upstream: add support for trailers in HTTP responses

2017-04-03 Thread Piotr Sikora via nginx-devel
Hey, > +static ngx_int_t > +ngx_http_upstream_copy_trailer(ngx_http_request_t *r, > +ngx_table_elt_t *h, ngx_uint_t offset) > +{ > +ngx_table_elt_t *ho; > + > +if (!r->upstream->conf->pass_trailers > +|| !r->allow_trailers || !r->expect_trailers) > +{ > +return NGX

[PATCH 2 of 3] Headers filter: add "add_trailer" directive

2017-04-03 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1490351854 25200 # Fri Mar 24 03:37:34 2017 -0700 # Node ID 5bab17ebe2b1f8ec42cf069bf484489c2a92c7a8 # Parent 8af81a0d66c0f69bcf501edcf10deed4c8f7fbd4 Headers filter: add "add_trailer" directive. Trailers added using this directive are evaluat

[PATCH 3 of 3] Upstream: add support for trailers in HTTP responses

2017-04-03 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1490351854 25200 # Fri Mar 24 03:37:34 2017 -0700 # Node ID 488c59bd49dcb1503144fe4d712165b69d1a5945 # Parent 5bab17ebe2b1f8ec42cf069bf484489c2a92c7a8 Upstream: add support for trailers in HTTP responses. Please note that due to how upstream m

[PATCH 1 of 3] HTTP: add support for trailers in HTTP responses

2017-04-03 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1490351854 25200 # Fri Mar 24 03:37:34 2017 -0700 # Node ID 8af81a0d66c0f69bcf501edcf10deed4c8f7fbd4 # Parent 39ff6939266e913e8bfd400e60f9520e70725a4d HTTP: add support for trailers in HTTP responses. Example: ngx_table_elt_t *h; h =

Re: [PATCH][bugfix] Upstream: clear the delayed flag to prevent blocking from sending.

2017-04-03 Thread 胡聪 (hucc)
Hi, On Sunday, Apr 2, 2017 9:35 PM +0300, Maxim Dounin wrote: >On Fri, Feb 17, 2017 at 11:18:48AM +0800, 胡聪 (hucc) wrote: > >> Hi, >> >> To reproduce the scene of send blocking, there must be more than >> two subrequests, and the more the better. >> > >Thank you for your report. > >After looking