Memory allocation for haproxy

2021-04-14 Thread Robert Ionescu
Hi all, I am working on a new patch for haproxy and would need the malloc method used to allocate memory for structs like the connection struct in connection-t.c In case I want to expand the struct with a new variable, do I need to take care of the memory for this variable or is there already a

Re: [PATCH] MINOR: sample: add json_string

2021-04-14 Thread Aleksandar Lazic
Hi. here now the current version of the patches. Regards Aleks. On 14.04.21 10:45, Aleksandar Lazic wrote: On 14.04.21 04:36, Willy Tarreau wrote: On Wed, Apr 14, 2021 at 03:02:20AM +0200, Aleksandar Lazic wrote: But then, could it make sense to also support "strict integers": values that

[OT PATCH 0/2] opentracing: use of the context name without prefix

2021-04-14 Thread Miroslav Zagorac
Hello all, the next two patches allow you to transfer the OpenTracing context via an HTTP headers without using a name prefix. This is very useful when transferring context from or to some other process that cannot add or remove a prefix from that data. Best regards, -- Zaga What can

[OT PATCH 1/2] MINOR: opentracing: correct calculation of the number of arguments in the args[]

2021-04-14 Thread Miroslav Zagorac
Hello all, It is possible that some arguments within the configuration line are not specified; that is, they are set to a blank string. For example: keyword '' arg_2 In that case the content of the args field will be like this: args[0]: 'keyword' args[1]:

[OT PATCH 2/2] MINOR: opentracing: transfer of context names without prefix

2021-04-14 Thread Miroslav Zagorac
Hello all, In order to enable the assignment of a context name, and yet exclude the use of that name (prefix in this case) when extracting the context from the HTTP header, a special character '-' has been added, which can be specified at the beginning of the prefix. So let's say if we look at

Re: Still 100% CPU usage in 2.3.9 & 2.2.13 (Was: Re: [2.2.9] 100% CPU usage)

2021-04-14 Thread Christopher Faulet
Le 10/04/2021 à 00:34, Robin H. Johnson a écrit : On Fri, Apr 09, 2021 at 10:14:26PM +0200, Christopher Faulet wrote: It seems you have a blocking call in one of your lua script. The threads dump shows many threads blocked in hlua_ctx_init. Many others are executing lua. Unfortunately, for a

[PATCH] MINOR: ist: Add `istclear(struct ist*)`

2021-04-14 Thread Tim Duesterhus
Christopher, Willy, On 4/13/21 6:34 PM, Christopher Faulet wrote: >> Thus I can't simply take a `struct ist*` for the destination, as an ist >> cannot communicate the size of the underlying buffer. I could >> technically take a `struct buffer`, but I'd still like the result to >> reside in an

Re: [PATCH] fix cirrus-ci builds by installing "pcre" package

2021-04-14 Thread Tim Düsterhus
Willy, On 4/14/21 7:00 PM, Илья Шипицин wrote: something changed in freebsd packaging. we now need to install pcre directly. This one looks good to me. Best regards Tim Düsterhus

Re: [PATCH] MINOR: sample: add json_string

2021-04-14 Thread Tim Düsterhus
Aleks, On 4/14/21 1:19 PM, Aleksandar Lazic wrote: From 46ddac8379324b645c662e19de39d5de4ac74a77 Mon Sep 17 00:00:00 2001 From: Aleksandar Lazic Date: Wed, 14 Apr 2021 13:11:26 +0200 Subject: [PATCH 2/2] MINOR: sample: converter: Add json_query converter With the json_query can a JSON value

[PATCH] fix cirrus-ci builds by installing "pcre" package

2021-04-14 Thread Илья Шипицин
Hello, something changed in freebsd packaging. we now need to install pcre directly. Ilya From 406a5b8cfee330cc74c18f0eca1811195e7eff6d Mon Sep 17 00:00:00 2001 From: Ilya Shipitsin Date: Wed, 14 Apr 2021 21:47:34 +0500 Subject: [PATCH] CI: cirrus: install "pcre" package it turned out that our

Re: [PATCH] MINOR: sample: add json_string

2021-04-14 Thread Aleksandar Lazic
On 14.04.21 18:41, Tim Düsterhus wrote: Aleks, On 4/14/21 1:19 PM, Aleksandar Lazic wrote: From 46ddac8379324b645c662e19de39d5de4ac74a77 Mon Sep 17 00:00:00 2001 From: Aleksandar Lazic Date: Wed, 14 Apr 2021 13:11:26 +0200 Subject: [PATCH 2/2] MINOR: sample: converter: Add json_query

Re: [PATCH] MINOR: ist: Add `istclear(struct ist*)`

2021-04-14 Thread Willy Tarreau
On Wed, Apr 14, 2021 at 07:14:30PM +0200, Tim Duesterhus wrote: > > You can pass a pointer on dst. In the normalizer, you can update its > > size. It is thus possible to use dst when calling > > http_replace_req_path() or http_replace_req_query(). > > > > I see, that makes sense to me. I

Re: [PATCH] MINOR: sample: add json_string

2021-04-14 Thread Willy Tarreau
On Wed, Apr 14, 2021 at 09:05:53AM +0200, Tim Düsterhus wrote: > Willy, > > On 4/14/21 4:36 AM, Willy Tarreau wrote: > > > How about "json_query" because it's exactly what it does :-) > > > > I'm not familiar with the notion of "query" to decode and extract contents > > but I'm not the most

Re: [PATCH] MINOR: sample: add json_string

2021-04-14 Thread Tim Düsterhus
Willy, On 4/14/21 4:36 AM, Willy Tarreau wrote: How about "json_query" because it's exactly what it does :-) I'm not familiar with the notion of "query" to decode and extract contents but I'm not the most representative user and am aware of the "jq" command- line utility that does this. So if

Re: [PATCH] MINOR: sample: add json_string

2021-04-14 Thread Aleksandar Lazic
On 14.04.21 04:36, Willy Tarreau wrote: On Wed, Apr 14, 2021 at 03:02:20AM +0200, Aleksandar Lazic wrote: But then, could it make sense to also support "strict integers": values that can accurately be represented as integers and which are within the JSON valid range for integers (-2^52 to 2^52