Re: [PATCH] BUG: payload: fix payload not retrieving arbitrary lengths

Ok, that's brilliant! Thanks a lot! Cheers, Felipe On 21 March 2017 at 11:48, Willy Tarreau <w...@1wt.eu> wrote: > On Tue, Mar 21, 2017 at 11:26:15AM -0300, Felipe Guerreiro Barbosa Ruiz > wrote: > > Hi Willy, > > > > Thanks for fixing it. I guess pasting from

Re: [PATCH] BUG: payload: fix payload not retrieving arbitrary lengths

can create a patch for each version, if necessary. Cheers, Felipe On 20 March 2017 at 03:30, Willy Tarreau <w...@1wt.eu> wrote: > Hi Felipe, > > On Thu, Mar 16, 2017 at 05:01:41PM -0300, Felipe Guerreiro Barbosa Ruiz > wrote: > > This fixes a regression introduced in d7

[PATCH] BUG: payload: fix payload not retrieving arbitrary lengths

This fixes a regression introduced in d7bdcb874bcb, that removed the ability to use req.payload(0,0) to read the whole buffer content. The offending commit is present starting in version 1.6, so the patch should be backported to versions 1.6 and 1.7. --- src/payload.c | 2 +- 1 file changed, 1

Re: Possible regression on HAProxy 1.6, related to ACLs and dynamic payload buffers

Sure thing, I'll get it tested and submit the patch. Thanks for the swift response. Cheers, Felipe On 15 March 2017 at 07:06, Willy Tarreau wrote: > On Wed, Mar 15, 2017 at 10:56:10AM +0100, Willy Tarreau wrote: > > > I did some digging and found that the ability to handle dynamic

Possible regression on HAProxy 1.6, related to ACLs and dynamic payload buffers

Hi all, After upgrading from 1.5 to 1.6 I noticed some ACLs stopped working. All of them looked like: acl some_name req.payload(0,0) <> I did some digging and found that the ability to handle dynamic buffers was added in 00f0084752eab236af80e61291d672e835790cff