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 toshikuni-fuk...@cybozu.co.jp wrote: Upstream: support named location for X-Accel-Redirect. Could please you provide usage examples?

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 calderon.tho...@gmail.com wrote: Hi Piotr, I was not aware that some efforts were ongoing to use PKCS#11 devices with nginx. However, my experience

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 pi...@cloudflare.com # Date 1414668641 25200 # Thu Oct 30 04:30:41 2014 -0700 # Node ID bb14c7659efb32d1d1f651bdf54a8c8157ef67f9 # Parent

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

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 mdou...@mdounin.ru 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

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 p...@cryptopro.ru 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

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 pi...@cloudflare.com # Date 1414668641 25200 # Thu Oct 30 04:30:41 2014 -0700 # Node ID bb14c7659efb32d1d1f651bdf54a8c8157ef67f9 # Parent

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 originally

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 toshikuni-fuk...@cybozu.co.jp # Date 1415098583 -32400 # Node ID 6f4517db02a8cd4068b9378bd93fe6290f54720d # Parent