Re: 2.0 regression? Control socket and ACL patterns with spaces

2019-09-12 Thread Willy Tarreau
On Thu, Sep 12, 2019 at 10:14:40AM +0200, Conrad Hoffmann wrote: > That's great to hear! I trust your judgement there, and it would > probably be the cleaner solution (as it re-enables the previous > behavior), but I'll just mention that I'd be perfectly fine with using > the payload syntax if

Re: 2.0 regression? Control socket and ACL patterns with spaces

2019-09-12 Thread Conrad Hoffmann
Hi, On Thu, 2019-09-12 at 08:12 +0200, Willy Tarreau wrote: > Hi Conrad, > > On Wed, Sep 11, 2019 at 04:53:18PM +0200, Conrad Hoffmann wrote: > > Hey again, > > > > sorry for reviving an old(ish) thread, but I would love to get some > > kind of feedback, even if it's just that this will stay

Re: 2.0 regression? Control socket and ACL patterns with spaces

2019-09-12 Thread Willy Tarreau
Hi Conrad, On Wed, Sep 11, 2019 at 04:53:18PM +0200, Conrad Hoffmann wrote: > Hey again, > > sorry for reviving an old(ish) thread, but I would love to get some > kind of feedback, even if it's just that this will stay this way and I > need to deal with it. > > I also tried with the payload

Re: 2.0 regression? Control socket and ACL patterns with spaces

2019-09-11 Thread Conrad Hoffmann
Hey again, sorry for reviving an old(ish) thread, but I would love to get some kind of feedback, even if it's just that this will stay this way and I need to deal with it. I also tried with the payload syntax in the meantime, but to no avail. Is there any supported way of adding patterns with

Re: 2.0 regression? Control socket and ACL patterns with spaces

2019-09-05 Thread Conrad Hoffmann
On Wed, 2019-09-04 at 21:41 +0200, Tim Düsterhus wrote: > Conrad, > > Am 04.09.19 um 10:17 schrieb Conrad Hoffmann: > > More for illustrative purposes than anything else I attached a > > patch > > that fixes the issue for me (against 2.0 repo). However, I have not > > yet > > spent a lot of time

Re: 2.0 regression? Control socket and ACL patterns with spaces

2019-09-04 Thread Tim Düsterhus
Conrad, Am 04.09.19 um 10:17 schrieb Conrad Hoffmann: > More for illustrative purposes than anything else I attached a patch > that fixes the issue for me (against 2.0 repo). However, I have not yet > spent a lot of time with the 2.0 code base, so I'll leave it up to you > to judge the quality