Re: [PATCH 2/4] MINOR: mailers, cfg: make time available for connection attempts to mailserver configurable with new 'timeout mail ' setting

2016-02-20 Thread P.Baauw
Hi Willy, Op 17-2-2016 om 10:27 schreef Willy Tarreau: Hi Pieter, for this one, could you please add an entry in the doc (configuration.txt) ? I've re-adjusted the commit message already, you can update this patch. Thanks, Willy Added a small configuration section. Regards, Pieter From

Re: [PATCH 1/4] mailer: increase default timeout to 10 seconds this allows the TCP connection to retry connecting when a packet is lost on the network

2016-02-16 Thread P.Baauw
Hi Willy, Simon, Op 16-2-2016 om 21:56 schreef Willy Tarreau: Simon, are you OK with this series from Pieter ? Should we backport them to 1.6 ? They look like fixes but I'm uncertain. Patches 1 and 4 i think should be backported to 1.6. 1- allows the tcp connection to send multiple syn

[PATCH] MINOR: resolvers, cfgparse, warn for incorrect 'timeout retry' keyword spelling

2016-02-16 Thread P.Baauw
Hi Willy, Baptiste, Hereby a 'backport' for 1.6. Hope its ok like this? I took a little different structure as i did for 1.7. But as there is no need to allow different keywords 'in the future' for 1.6 i think it is ok. Regards, Pieter From a8346de18704ef283621aabf4cb2edc3ebb0ce6d Mon Sep 17

Re: [PATCH 3/4] MEDIUM: resolvers, cfgparse, do not allow 'timeout retry' to be misspelled

2016-02-15 Thread P.Baauw
Hi Willy, Op 15-2-2016 om 19:46 schreef Willy Tarreau: Hi Pieter, On Mon, Feb 15, 2016 at 07:40:42PM +0100, P.Baauw wrote: Hi Willy, I decided to make it MEDIUM because it might break an existing config that is using a 'wrong' config option and would currently be parsed and working 'properly

Re: [PATCH 3/4] MEDIUM: resolvers, cfgparse, do not allow 'timeout retry' to be misspelled

2016-02-15 Thread P.Baauw
Hi Willy, I decided to make it MEDIUM because it might break an existing config that is using a 'wrong' config option and would currently be parsed and working 'properly' when configured like this: timeout retri 1s or timeout WRONG 1s Would both work 'properly' and be used for

[PATCH 2/4] MINOR: mailers, cfg: make time available for connection attempts to mailserver configurable with new 'timeout mail ' setting

2016-02-13 Thread P.Baauw
From f97319506e74e912cb77fa35a3e9d1618b8130f2 Mon Sep 17 00:00:00 2001 From: Pieter Baauw Date: Sat, 13 Feb 2016 15:33:40 +0100 Subject: [PATCH 2/4] MINOR: mailers, cfg: make time available for connection attempts to mailserver configurable with new 'timeout mail '

[PATCH 1/4] mailer: increase default timeout to 10 seconds this allows the TCP connection to retry connecting when a packet is lost on the network

2016-02-13 Thread P.Baauw
From 4583d6c5a96413bc6b827863b2be8e0c5b68910b Mon Sep 17 00:00:00 2001 From: Pieter Baauw Date: Fri, 12 Feb 2016 14:35:20 +0100 Subject: [PATCH 1/4] mailer: increase default timeout to 10 seconds this allows the TCP connection to retry connecting when a packet is lost

[PATCH 4/4] MINOR: mailers, use for all line endings

2016-02-13 Thread P.Baauw
From 029156e2e2c6d59463e5dd4b9b574a18fdf117b7 Mon Sep 17 00:00:00 2001 From: Pieter Baauw Date: Sat, 13 Feb 2016 16:27:35 +0100 Subject: [PATCH 4/4] MINOR: mailers, use for all line endings Not doing so causes issues with Exchange2013 not processing the message body

[PATCH 3/4] MEDIUM: resolvers, cfgparse, do not allow 'timeout retry' to be misspelled

2016-02-13 Thread P.Baauw
From 83f908683a137b3e947e7d12b7e90f1b4a22db58 Mon Sep 17 00:00:00 2001 From: Pieter Baauw Date: Sat, 13 Feb 2016 15:51:58 +0100 Subject: [PATCH 3/4] MEDIUM: resolvers, cfgparse, do not allow 'timeout retry' to be misspelled --- src/cfgparse.c | 32