Re: Set-Cookie Secure

2017-10-05 Thread Igor Cicimov
Hi, On Fri, Oct 6, 2017 at 2:50 AM, mlist wrote: > Hi Igor, some news about this ? > > > > *From:* mlist > *Sent:* venerdì 22 settembre 2017 08:58 > *To:* 'Igor Cicimov' > *Cc:* 'HAProxy' > *Subject:* RE: Set-Cookie

Re: another cppcheck finding

2017-10-05 Thread Baptiste
I also fixed it in a patch set to make the resolution pool dynamic :) Baptiste

Re: [PATCHES] 51d: fix warning when building with 51Degrees release version 3.2.12.12

2017-10-05 Thread Willy TARREAU
On Thu, Oct 05, 2017 at 04:09:09PM +, Ben Shillito wrote: > Hi Willy, > > I have pulled the latest master, but I only see two commits from today. I > assume I'm in the right place " http://git.haproxy.org/git/haproxy;? Argh! Sorry, my fault, I forgot to push :-( And as stupid as I am I did

RE: [PATCHES] 51d: fix warning when building with 51Degrees release version 3.2.12.12

2017-10-05 Thread Ben Shillito
Hi Willy, I have pulled the latest master, but I only see two commits from today. I assume I'm in the right place " http://git.haproxy.org/git/haproxy;? Using 3.2.10 in 1.7 is fine for Pattern, my only concern is that the old Trie source is not available to for users to compile in 3.2.10. All

RE: Set-Cookie Secure

2017-10-05 Thread mlist
Hi Igor, some news about this ? From: mlist Sent: venerdì 22 settembre 2017 08:58 To: 'Igor Cicimov' Cc: 'HAProxy' Subject: RE: Set-Cookie Secure I have acl to leave some sites http (not redirected to https), so adding secure flag on

Re: [PATCHES] 51d: fix warning when building with 51Degrees release version 3.2.12.12

2017-10-05 Thread Willy TARREAU
On Thu, Oct 05, 2017 at 03:15:20PM +, Ben Shillito wrote: > Yes of course. I have attached a patch which has the correct branch and > updated instructions about where to get the free Hash Trie file now that it > is no longer part of the git repository. But this one is for the doc before

RE: [PATCHES] 51d: fix warning when building with 51Degrees release version 3.2.12.12

2017-10-05 Thread Ben Shillito
Hi Willy, Yes of course. I have attached a patch which has the correct branch and updated instructions about where to get the free Hash Trie file now that it is no longer part of the git repository. Regards, Ben Shillito Developer O: +44 1183 287152 E: b...@51degrees.com T: @51Degrees

RE: [PATCHES] 51d: fix warning when building with 51Degrees release version 3.2.12.12

2017-10-05 Thread Ben Shillito
Hi Dragan, The changes seem reasonable to me. A couple of things I should mention though. The branch name for 3.2.12.12 should be in the same format as 3.2.10 (i.e. "v3.2.12"). The other thing is that the statement about 3.2.10 is not correct. The Trie data files were large and the solution

Re: [PATCH] Add info about stats report when a reload is done in management.txt

2017-10-05 Thread Lukas Tribus
Hello Ricardo, Am 05.10.2017 um 09:47 schrieb Ricardo Fraile: > +Note that when a "restart" is doing, the new process have the listening ports > +but the old process continue with the existing connections until they close. > +All the active connections that the old process did are still working

Re: Inquiry: Information, HAProxy

2017-10-05 Thread Lukas Tribus
Hello Logan, Am 05.10.2017 um 02:29 schrieb Logan Hicks: > > Aaron West, > > Thank you! Its great to know I'm not just tossing information into oblivion. > Thank you for such a fast reply. > > All, > > > I am looking for a good tutorial to help me truly understand how and why > HAProxy works.

RE: Question related to gpc0_rate values in stick-table

2017-10-05 Thread Saurabh Patwardhan
Thanks Lukas. Can you please provide the details of the known issue which might be related to this situation? I have gone through the list of all unknown issues but couldn't find the relevant one. Regards, Saurabh -Original Message- From: Lukas Tribus [mailto:lu...@gmx.net] Sent:

Re: [PATCHES] 51d: fix warning when building with 51Degrees release version 3.2.12.12

2017-10-05 Thread Willy Tarreau
Hi Dragan, On Mon, Oct 02, 2017 at 11:14:12AM +0200, Dragan Dosen wrote: > Hi all, > > I'm sending you a new set of patches -- fix for warning that appears > when building with 51Degrees release that uses a new Hash Trie algorithm > (release version 3.2.12.12): > > * BUILD/MINOR: 51d: fix

Re: [PATCH] BUG/MEDIUM: http: Return an error when url_dec sample converter failed

2017-10-05 Thread Willy TARREAU
On Thu, Oct 05, 2017 at 11:04:36AM +0200, Christopher Faulet wrote: > Le 05/10/2017 à 10:52, Christopher Faulet a écrit : > >Hi, > > > >Here is the patch that fixes the bug reported by Marcus (see "Haproxy > >segfault error 4 in libc-2.24"). > > > > Sorry, here is a new version of my patch. No

Re: [PATCH] BUG/MEDIUM: http: Return an error when url_dec sample converter failed

2017-10-05 Thread Christopher Faulet
Le 05/10/2017 à 10:52, Christopher Faulet a écrit : Hi, Here is the patch that fixes the bug reported by Marcus (see "Haproxy segfault error 4 in libc-2.24"). Sorry, here is a new version of my patch. No reason to consider zero-length string as an error. -- Christopher Faulet >From

[PATCH] BUG/MEDIUM: http: Return an error when url_dec sample converter failed

2017-10-05 Thread Christopher Faulet
Hi, Here is the patch that fixes the bug reported by Marcus (see "Haproxy segfault error 4 in libc-2.24"). Thanks -- Christopher Faulet >From 077217437a09e5d81216d377d9aff73dc1ce7122 Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Thu, 5 Oct 2017 10:03:12 +0200

Re: Haproxy segfault error 4 in libc-2.24

2017-10-05 Thread Christopher Faulet
Le 04/10/2017 à 05:12, Willy Tarreau a écrit : On Wed, Oct 04, 2017 at 05:07:07AM +0200, Willy Tarreau wrote: On Wed, Oct 04, 2017 at 04:40:53AM +0200, Willy Tarreau wrote: On Tue, Oct 03, 2017 at 06:57:45PM +0200, Marcus Ulbrich wrote: Hey Jarno, This seems to work stable! The idea for this

[PATCH] Add info about stats report when a reload is done in management.txt

2017-10-05 Thread Ricardo Fraile
Hi, It's interesting to have in the section "4. Stopping and restarting HAProxy" in the management.txt document some information related to the behaviour of the stats when a restart is done. As suggestion, here is my patch. Thanks, Ricardo F. diff --git a/doc/management.txt