Re: [PATCH] BUG/Minor : allow to log cookie for tarpit and denied request

[Bertrand Paquet]

Hi,

Please find the patch attached.

Regards,

Bertrand

On Wed, Apr 6, 2016 at 12:59 PM, Willy Tarreau <w...@1wt.eu> wrote:

> Hi Bertrand,
>
> On Wed, Apr 06, 2016 at 11:58:31AM +0200, Bertrand Paquet wrote:
> > Hi all,
> >
> > The following patch allow to log cookie for tarpit and denied request.
> > This minor bug affect at least 1.5, 1.6 and 1.7 branch.
>
> Thank you.
>
> Please check below, it seems that your mailer has mangled spaces/tabs :
>
> > diff --git a/src/proto_http.c b/src/proto_http.c
> > index 0c37736..6ab14ce 100644
> > --- a/src/proto_http.c
> > +++ b/src/proto_http.c
> > @@ -4258,6 +4258,12 @@ int http_process_req_common(struct stream *s,
> struct
> > channel *req, int an_bit, s
> >    * if the client closes first.
> >    */
> >   channel_dont_connect(req);
> > +
> > + /* Allow cookie logging
> > +  */
> > + if (s->be->cookie_name || sess->fe->capture_name)
> > +   manage_client_side_cookies(s, req);
> > +
>
> (...)
>
> If you're having trouble fixing it, just send the patch as an attachment
> and
> it should be OK.
>
> Thanks!
> Willy
>

tarpit.patch
Description: Binary data