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

2017-04-21 Thread Maxim Dounin
Hello! On Mon, Apr 03, 2017 at 02:32:13AM -0700, Piotr Sikora via nginx-devel wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1490351854 25200 > # Fri Mar 24 03:37:34 2017 -0700 > # Node ID 488c59bd49dcb1503144fe4d712165b69d1a5945 > # Parent 5bab17ebe2b1f8ec42cf069bf484489c2a9

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

2017-04-21 Thread Maxim Dounin
Hello! On Mon, Apr 03, 2017 at 02:32:12AM -0700, Piotr Sikora via nginx-devel wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1490351854 25200 > # Fri Mar 24 03:37:34 2017 -0700 > # Node ID 5bab17ebe2b1f8ec42cf069bf484489c2a92c7a8 > # Parent 8af81a0d66c0f69bcf501edcf10deed4c8f

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

2017-04-21 Thread Maxim Dounin
Hello! On Mon, Apr 03, 2017 at 02:32:11AM -0700, Piotr Sikora via nginx-devel wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1490351854 25200 > # Fri Mar 24 03:37:34 2017 -0700 > # Node ID 8af81a0d66c0f69bcf501edcf10deed4c8f7fbd4 > # Parent 39ff6939266e913e8bfd400e60f9520e707

Subrequest Cycle

2017-04-21 Thread Antonio Nappa
Hello, I would like to trigger a subrequest that will call a location that has an upstream. What I am experimenting is a loop of subrequests, also if I do finalize on the subrequest in the subsequent calls to the filter. if (r != r->main) { /* subrequest */ ngx_http_finalize_request(r, NGX_

Re: [PATCH] Contrib: vim syntax, support block region

2017-04-21 Thread Maxim Dounin
Hello! On Fri, Apr 21, 2017 at 04:39:03PM +0800, OOO wrote: > Actually nginx.vim supports on and off[1] before. > And some option also been highlighted because there are directives > with the same name, ex: ssl. > So if we complete remove this. > It will looks like the function is missing. Yes,

Re: [PATCH] Contrib: vim syntax, support block region

2017-04-21 Thread OOO
Actually nginx.vim supports on and off[1] before. And some option also been highlighted because there are directives with the same name, ex: ssl. So if we complete remove this. It will looks like the function is missing. [1]:https://github.com/nginx/nginx/blob/73d27510c0d7022384a611269af22ff01634c