Re: Proposed changeset to fix client cert from ngx_ssl_get_certificate passed as HTTP header value

2016-02-10 Thread Sam McKelvie
OK, sorry for the delay. Here is a changeset that implements $ssl_client_escaped_certificate, following your latest comments. It works well with node/express. OK with this? ~Sam # HG changeset patch # User Sam McKelvie # Date 1455143619 28800 # Wed Feb 10 14:33:39 2016 -0800 # Node ID f3f

Re: [PATCH] Limit conn: Added "off" parameter to the 'limit_conn' directive

2016-02-10 Thread Maxim Dounin
Hello! On Wed, Feb 10, 2016 at 11:31:42AM +0600, Pavel V. wrote: > Hi, Maxim! > > Thanks for reviewing my patches. > > >> + > >> +if (ngx_strcmp(value[1].data, "off") == 0) { > >> +if (cf->args->nelts != 2) { > >> +return "has invalid number of arguments"; > >> +

[nginx] Dynamic modules: added auth_basic module libs via ngx_mo...

2016-02-10 Thread Ruslan Ermilov
details: http://hg.nginx.org/nginx/rev/965e4223e702 branches: changeset: 6391:965e4223e702 user: Ruslan Ermilov date: Wed Feb 10 11:36:25 2016 +0300 description: Dynamic modules: added auth_basic module libs via ngx_module_libs=. diffstat: auto/modules | 3 +-- 1 files changed, 1

Re: [PATCH] Limit conn: Added "off" parameter to the 'limit_conn' directive

2016-02-10 Thread Valentin V. Bartenev
On Wednesday 10 February 2016 00:09:09 Pavel V. Rochnyack wrote: > # HG changeset patch > # User Pavel V. Rochnyack > # Date 1454835814 -21600 > # Node ID b4748ebdd06ba79aa27e0c54fc1d627d13966bed > # Parent 3577c021f21eb4de6d09c1d624ba77ee9ee1eb6d > Limit conn: Added "off" parameter to the 'limit