On Fri, Dec 28, 2012 at 03:03:39AM +0100, David BERARD wrote: > Hi Willy, > > 2012/12/28 Willy Tarreau <w...@1wt.eu>: > > Hi David, > > > > On Fri, Dec 28, 2012 at 01:46:32AM +0100, David BERARD wrote: > >> Hi, > >> > >> I start to evaluate the new add-header/set-header option, I encountered > >> issues. > >> I use nginx as web server behind HAProxy, and nginx reply 400 bad > >> request at every request. > >> > >> The header set with add-header/set-header is malformed, I have a null > >> character before the end of the > >> line, for exemple : > >> > >> http-request set-header X-SSL-TOTO %[ssl_fc] > >> > >> set the following header : X-SSL-TOTO: 1\000\r\n > >> > >> The length of header value seem too big, the following workaround fix > >> this issue (not optimal I now): > > > > OK I see what's happening. I'd rather fix the root cause of this issue, > > it's build_logline() which returns an ambiguous length. > > > > Could you please try the attached patch instead ? It works for me right > > here. > > Works for me too. Thanks :)
Thanks for the quick feedback, I'm pushing it then. Have a good night, Willy