Thanks! On Tue, Apr 28, 2015 at 2:07 PM, Willy Tarreau <w...@1wt.eu> wrote: > Hi Andrew, > > On Tue, Apr 28, 2015 at 10:54:58AM -0500, Andrew Hayworth wrote: > (...) >> I changed %HR -> %HU, and mentioned '(path)' in the docs. > > I found it was not changed in the parser nor the doc but I fixed it, don't > worry. > >> > Now concerning the bug, it's not very important but it does exist : >> >> Good catch - Suffice it to say I've tested this patch a bit more thoroughly. >> :-) >> >> After a little reflection and during testing, I also found it >> confusing that the new >> directives sometimes logged "<BADREQ>" and sometimes parsed as much as >> they could out of the request line before giving up. This latest patch >> more consistently >> logs "<BADREQ>" if the line is bad. I ended up counting spaces, based on the >> premise that any valid request in any version of the protocol must >> contain at least >> one whitespace character. > > That's a good idea indeed. > > Thanks for your patience, I've now merged your work. That will definitely > be useful in a number of circumstances. > > Cheers, > Willy >
-- - Andrew Hayworth
