Re: unique-id-header set twice

2016-04-28 Thread Willy Tarreau
On Fri, Apr 22, 2016 at 04:37:04PM +0200, Erwin Schliske wrote: > Hello, > > for some of our services requests pass haproxy twice. As we have set the > global option unique-id-header this header is added twice. > > Is there a way to avoid this? I could set the header with 'http-request >

Re: Regarding client side keep-alive

2016-04-28 Thread Willy Tarreau
On Fri, Apr 22, 2016 at 02:48:43PM +0200, Baptiste wrote: > > Basically, I want the client<>haProxy side to use keep alive, but > > haProxy<>server side to close every connection and open a new one for each > > request. > > > > Is it then correct to use http-server-close? > > You are correct.

Re: haproxy and pcre

2016-04-28 Thread Willy Tarreau
On Thu, Apr 28, 2016 at 02:36:44PM -0400, CJ Ess wrote: > I'm wanting to make a case for compiling haproxy against a modern version > of pcre (supporting jit) when we roll out the next release to my day job. > Anyone have some numbers handy that show the benefit of doing so? Please check these

Re: Haproxy running on 100% CPU and slow downloads

2016-04-28 Thread Sachin Shetty
Thanks Lukas and Willy. I am in the process of getting 1.6.4-20160426 deployed in our QA, I will keep you guys posted. On 4/29/16, 11:09 AM, "Willy Tarreau" wrote: >Hi guys, > >On Tue, Apr 26, 2016 at 08:46:37AM +0200, Lukas Tribus wrote: >> Hi Sachin, >> >> >> there is another

Re: Haproxy running on 100% CPU and slow downloads

2016-04-28 Thread Willy Tarreau
Hi guys, On Tue, Apr 26, 2016 at 08:46:37AM +0200, Lukas Tribus wrote: > Hi Sachin, > > > there is another fix Willy recently committed, its ff9c7e24fb [1] > and its in the snapshots [2] since 1.6.4-20160426. > > This is supposed to fix the issue altogether. > > Please let us know if this

Re: [PATCH] BUG/MINOR: log: fix a typo that would cause %HP to log

2016-04-28 Thread Willy Tarreau
On Tue, Apr 26, 2016 at 01:39:02AM +0200, Nenad Merdanovic wrote: > Typo was introduced in 57bc891 ("BUG/MEDIUM: log: fix risk of > segfault when logging HTTP fields in TCP mode") which inverted the > condition in the test and caused to be logged when using > %HP. G... my bad. Thanks for

Re: Support for Keep-Alive header and timeouts

2016-04-28 Thread Willy Tarreau
Hi Lukas, On Mon, Apr 25, 2016 at 07:29:26PM +0200, Lukas Tribus wrote: > You are saying force closing the session in the middle of a response leading > to > a truncate response and forcing the client to do the request all over again > is > something you would consider for your production

Re: [PATCH] BUG/MINOR: dumpstats: Fix the "Total bytes saved" counter in, backends stats

2016-04-28 Thread Willy Tarreau
On Thu, Apr 28, 2016 at 03:57:09PM +0200, Christopher Faulet wrote: > Hi, > > This is just a typo fix. good catch, thanks Christopher, willy

Re: PATCH 1/2: BUG/MEDIUM lua argument boundary fix

2016-04-28 Thread Willy Tarreau
On Thu, Apr 28, 2016 at 03:35:14PM +0200, Thierry FOURNIER wrote: > Hi, > > Thanks for the patch. > Willy, you can apply these two pathes. Both patches applied, thanks guys. David, I'll backport them to 1.6 ASAP. Cheers, willy

haproxy and pcre

2016-04-28 Thread CJ Ess
I'm wanting to make a case for compiling haproxy against a modern version of pcre (supporting jit) when we roll out the next release to my day job. Anyone have some numbers handy that show the benefit of doing so?

Re: PATCH 1/2: BUG/MEDIUM lua argument boundary fix

2016-04-28 Thread David CARLIER
Nice, in case I forgot to mention, would be kind to backport the first in 1.6 branch. Thanks. Kindest regards. On 28 April 2016 at 14:35, Thierry FOURNIER wrote: > Hi, > > Thanks for the patch. > Willy, you can apply these two pathes. > > Thierry > > On Wed, 27 Apr 2016

[PATCH] BUG/MINOR: dumpstats: Fix the "Total bytes saved" counter in, backends stats

2016-04-28 Thread Christopher Faulet
Hi, This is just a typo fix. -- Christopher Faulet >From bfc2be71794987fcfb0b5806607617431e23a65d Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Thu, 28 Apr 2016 15:09:31 +0200 Subject: [PATCH] BUG/MINOR: dumpstats: Fix the "Total bytes saved" counter in backends

Re: PATCH 1/2: BUG/MEDIUM lua argument boundary fix

2016-04-28 Thread Thierry FOURNIER
Hi, Thanks for the patch. Willy, you can apply these two pathes. Thierry On Wed, 27 Apr 2016 16:27:16 +0100 David CARLIER wrote: > Hi all, > > This is a first patch for the lua module already overviewed by Thierry > Fournier. > This one concerns a bug fix which was