Re: req.cook_cnt() broken?

2017-08-25 Thread Willy Tarreau
Hi Daniel, On Fri, Aug 25, 2017 at 12:47:41PM +0200, Daniel Schneller wrote: > On 24. Aug. 2017, at 01:50, Cyril Bonté wrote: > > > > You're right. currently, the code and the documentation don't say the same > > things. > > > > Can you try the attached patch ? > > > >

Re: req.cook_cnt() broken?

2017-08-25 Thread Daniel Schneller
On 24. Aug. 2017, at 01:50, Cyril Bonté wrote: > > You're right. currently, the code and the documentation don't say the same > things. > > Can you try the attached patch ? > > -- > Cyril Bonté > Thanks for the patch! Tried against 1.8, 1.7.9, and 1.6.13 just now.

Re: req.cook_cnt() broken?

2017-08-23 Thread Cyril Bonté
Hi Daniel, Le 23/08/2017 à 11:50, Daniel Schneller a écrit : Kindly bumping this during the summer vacation time for potentially new recipients :) On 21. Aug. 2017, at 21:14, Daniel Schneller wrote: Hi! According to the documentation req.cook_cnt([])

Re: req.cook_cnt() broken?

2017-08-23 Thread Daniel Schneller
Kindly bumping this during the summer vacation time for potentially new recipients :) > On 21. Aug. 2017, at 21:14, Daniel Schneller > wrote: > > Hi! > > According to the documentation > > req.cook_cnt([]) : integer > Returns an integer value

req.cook_cnt() broken?

2017-08-21 Thread Daniel Schneller
Hi! According to the documentation req.cook_cnt([]) : integer Returns an integer value representing the number of occurrences of the cookie in the request, or all cookies if is not specified. it should be possible to do something like this to reject a request if it contains more than