[PATCH] BUG/MINOR: server: fix slowstart behavior

2024-04-09 Thread Damien Claisse
We observed that a dynamic server which health check is down for longer than slowstart delay at startup doesn't trigger the warmup phase, it receives full traffic immediately. This has been confirmed by checking haproxy UI, weight is immediately the full one (e.g. 75/75), without any throttle

[PATCH] BUG/MINOR: server: fix persistence cookie for dynamic servers

2024-03-27 Thread Damien Claisse
When adding a server dynamically, we observe that when a backend has a dynamic persistence cookie, the new server has no cookie as we receive the following HTTP header: set-cookie: test-cookie=; Expires=Thu, 01-Jan-1970 00:00:01 GMT; path=/ Whereas we were expecting to receive something like the

Re: [PATCH] BUG/MINOR: server: fix persistence cookie for dynamic servers

2024-03-22 Thread Damien Claisse
Hi Amaury! Sorry for the HTML message, I have to use a *** well-known enterprise MUA :( Le 22/03/2024 09:09, « Amaury Denoyelle » a écrit : This patch raises several interrogations. First, dynamic servers are currently intended to not support cookies, hence why the keyword is disabled for

[PATCH] BUG/MINOR: server: fix persistence cookie for dynamic servers

2024-03-21 Thread Damien Claisse
When adding a server dynamically, we observe that when a backend has a dynamic persistence cookie, the new server has no cookie as we receive the following HTTP header: set-cookie: test-cookie=; Expires=Thu, 01-Jan-1970 00:00:01 GMT; path=/ Whereas we were expecting to receive something like the

[PATCH 1/1] MINOR: log: Add "Tu" timer

2020-04-28 Thread Damien Claisse
It can be sometimes useful to measure total time of a request as seen from an end user, including TCP/TLS negotiation, server response time and transfer time. "Tt" currently provides something close to that, but it also takes client idle time into account, which is problematic for keep-alive

[PATCH 0/1] *** Add Tu timer ***

2020-04-28 Thread Damien Claisse
Hi Willy, Thanks you very much for your answer. Let's go with Tu, sounds relevant to me. I've also slightly updated documentation, please tell me if this is not clear enough. Cheers, Damien Damien Claisse (1): MINOR: log: Add "Tu" timer doc/configuration.txt | 11 +++ inc

Re: [PATCH 0/2] *** Add TT timer ***

2020-04-16 Thread Damien Claisse
Hi Willy, Le 16/04/2020 17:00, « Willy Tarreau » a écrit : I'm a bit confused by how it can be efficiently used. Because it still includes the client-side handshake which only happens before the first request, so if you're interested in removing idle time from keep-alive

[PATCH 0/2] *** Add TT timer ***

2020-04-16 Thread Damien Claisse
. Damien Claisse (2): MINOR: log: rename LOG_FMT_TT constant MINOR: log: Add "TT" timer doc/configuration.txt | 8 include/types/log.h | 1 + src/log.c | 16 ++-- 3 files changed, 23 insertions(+), 2 deletions(-) -- 2.20.1

[PATCH 1/2] MINOR: log: rename LOG_FMT_TT constant

2020-04-16 Thread Damien Claisse
Rename it to LOG_FMT_Tt (same case as Tt timer), to reserve LOG_FMT_TT for another usage. --- include/types/log.h | 2 +- src/log.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/include/types/log.h b/include/types/log.h index c348caa1e..3720fe73f 100644 ---

[PATCH 2/2] MINOR: log: Add "TT" timer

2020-04-16 Thread Damien Claisse
It can be sometimes useful to measure total time of a request, including TCP/TLS negotiation, server response time and transfer time. "Tt" currently provides something close to that, but it also takes client idle time into account, which is problematic for keep-alive requests as idle time can be

[PATCH V4] MINOR: sample: add us/ms support to date/http_date

2019-10-30 Thread Damien Claisse
It can be sometimes interesting to have a timestamp with a resolution of less than a second. It is currently painful to obtain this, because concatenation of date and date_us lead to a shorter timestamp during first 100ms of a second, which is not parseable and needs ugly ACLs in configuration to

[PATCH V3] MINOR: sample: add us/ms support to date sample

2019-10-28 Thread Damien Claisse
It can be sometimes interesting to have a timestamp with a resolution of less than a second. It is currently painful to obtain this, because concatenation of date and date_us lead to a shorter timestamp during first 100ms of a second, which is not parseable and needs ugly ACLs in configuration to

[PATCH V2] MINOR: sample: add us/ms support to date sample

2019-10-28 Thread Damien Claisse
It can be sometimes interesting to have a timestamp with a resolution of less than a second. It is currently painful to obtain this, because concatenation of date and date_us lead to a shorter timestamp during first 100ms of a second, which is not parseable and needs ugly ACLs in configuration to

[PATCH] MINOR: sample: add us/ms support to date sample

2019-10-24 Thread Damien Claisse
It can be sometimes interesting to have a timestamp with a resolution of less than a second. It is currently painful to obtain this, because concatenation of date and date_us lead to a shorter timestamp during the first 100ms of a second, which is not parseable and needs ugly ACLs in configuration