Re: [PATCH] Upstream: support named location for X-Accel-Redirect

2014-11-10 Thread Valentin V. Bartenev
On Monday 10 November 2014 09:54:13 Toshikuni Fukaya wrote: > Hi, > > (2014/11/08 0:59), Igor Sysoev wrote: > > On 04 Nov 2014, at 14:18, Toshikuni Fukaya > > wrote: > > > >> Upstream: support named location for X-Accel-Redirect. > > > > Could please you provide usage examples? > > > > > > Here

Re: [PATCH] Upstream: support named location for X-Accel-Redirect

2014-11-10 Thread Igor Sysoev
On 10 Nov 2014, at 03:54, Toshikuni Fukaya wrote: > (2014/11/08 0:59), Igor Sysoev wrote: >> On 04 Nov 2014, at 14:18, Toshikuni Fukaya >> wrote: >> >>> Upstream: support named location for X-Accel-Redirect. >> >> Could please you provide usage examples? >> >> > > Here is my (simplified)

Re: [PATCH] Add PKCS#11 support to nginx http module

2014-11-10 Thread Thomas Calderon
Hi all, Is someone else interested in providing feedback for my patch ? Regards, Thomas. On Mon, Nov 3, 2014 at 11:30 PM, Thomas Calderon wrote: > Hi Piotr, > > I was not aware that some efforts were ongoing to use PKCS#11 devices with > nginx. > However, my experience with OpenSSL engine sup

Re: [PATCH] Upstream: add "proxy_ssl_certificate" and friends

2014-11-10 Thread Maxim Dounin
Hello! On Fri, Nov 07, 2014 at 03:26:43PM -0800, Piotr Sikora wrote: > Hey, > > > # HG changeset patch > > # User Piotr Sikora > > # Date 1414668641 25200 > > # Thu Oct 30 04:30:41 2014 -0700 > > # Node ID bb14c7659efb32d1d1f651bdf54a8c8157ef67f9 > > # Parent 87ada3ba1392fadaf4d9193b5d345

Re: [PATCH] Add PKCS#11 support to nginx http module

2014-11-10 Thread Maxim Dounin
Hello! On Mon, Nov 10, 2014 at 03:54:20PM +0100, Thomas Calderon wrote: > Hi all, > > Is someone else interested in providing feedback for my patch ? Dmitrii's patch is currently a primary candidate for inclusion. I agree with Piotr - it looks much better as it doesn't introduce additional d

Re: [PATCH] Core: parse octal/hexadecimal numeric config directives

2014-11-10 Thread Maxim Dounin
Hello! On Fri, Nov 07, 2014 at 01:19:43PM -0800, Ari Aosved wrote: > # HG changeset patch > # User Ari Aosved > # Date 1415392382 28800 > # Fri Nov 07 12:33:02 2014 -0800 > # Node ID 742fb1a59b705ba71f2bab5fede58ff5885ebc71 > # Parent 234c5ecb00c04c67bcbb4a3be45fd844f2289462 > Core: parse

Re: [PATCH] Add PKCS#11 support to nginx http module

2014-11-10 Thread Thomas Calderon
Hi Maxim, On Mon, Nov 10, 2014 at 4:11 PM, Maxim Dounin wrote: > Hello! > > On Mon, Nov 10, 2014 at 03:54:20PM +0100, Thomas Calderon wrote: > > > Hi all, > > > > Is someone else interested in providing feedback for my patch ? > > Dmitrii's patch is currently a primary candidate for inclusion.

Re: [PATCH] Add PKCS#11 support to nginx http module

2014-11-10 Thread Dmitrii Pichulin
Are you sure that Dmitrii's patch will allow to use dedicated key-pairs for each site declaration. It was tested, example: http://mailman.nginx.org/pipermail/nginx-devel/2014-October/006151.html ___ nginx-devel mailing list nginx-devel@nginx.org ht

Re: [PATCH] Add PKCS#11 support to nginx http module

2014-11-10 Thread Thomas Calderon
On Mon, Nov 10, 2014 at 4:50 PM, Dmitrii Pichulin wrote: > >> Are you sure that Dmitrii's patch will allow to use dedicated key-pairs >> for each site declaration. >> >> > It was tested, example: > > http://mailman.nginx.org/pipermail/nginx-devel/2014-October/006151.html Indeed, but the example

Re: [PATCH] Upstream: add "proxy_ssl_certificate" and friends

2014-11-10 Thread Maxim Dounin
Hello! On Thu, Oct 30, 2014 at 04:31:37AM -0700, Piotr Sikora wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1414668641 25200 > # Thu Oct 30 04:30:41 2014 -0700 > # Node ID bb14c7659efb32d1d1f651bdf54a8c8157ef67f9 > # Parent 87ada3ba1392fadaf4d9193b5d345c248be32f77 > Upstream

Re: [PATCH] Upstream: support named location for X-Accel-Redirect

2014-11-10 Thread Maxim Dounin
Hello! On Tue, Nov 04, 2014 at 08:18:44PM +0900, Toshikuni Fukaya wrote: > # HG changeset patch > # User Toshikuni Fukaya > # Date 1415098583 -32400 > # Node ID 6f4517db02a8cd4068b9378bd93fe6290f54720d > # Parent dff86e2246a53b0f4a61935cd5c8c0a0f66d0ca2 > Upstream: support named location for X-

Re: [PATCH] Upstream: add "proxy_ssl_certificate" and friends

2014-11-10 Thread Piotr Sikora
Hey Maxim, > I think that it would be better to preserve current style used in > the proxy module by placing configuration directive handling into > the block with other configuration directives, like this: Looks good, thanks. Best regards, Piotr Sikora _

Re: Gzip Gunzip: always flush busy bufs when the incoming chain is NULL.

2014-11-10 Thread Yichun Zhang (agentzh)
Hello! On Wed, Nov 5, 2014 at 5:08 PM, Yichun Zhang (agentzh) wrote: > Sorry again, it actually checked "c->buffered & > NGX_HTTP_LOWLEVEL_BUFFERED". This condition is indeed too strong and > I've made it check its own busy bufs instead. > Hmm, the problem here is more complicated than I original

Re: [PATCH] Upstream: support named location for X-Accel-Redirect

2014-11-10 Thread Toshikuni Fukaya
Hi, (2014/11/11 3:02), Maxim Dounin wrote: Hello! On Tue, Nov 04, 2014 at 08:18:44PM +0900, Toshikuni Fukaya wrote: # HG changeset patch # User Toshikuni Fukaya # Date 1415098583 -32400 # Node ID 6f4517db02a8cd4068b9378bd93fe6290f54720d # Parent dff86e2246a53b0f4a61935cd5c8c0a0f66d0ca2 Upst