Hi Aleks, On Sun, Aug 02, 2015 at 06:55:26PM +0200, Aleksandar Lazic wrote: > Hi Andrew. > > Am 31-07-2015 18:21, schrieb Andrew Hayworth: > >Since this came up in another thread, it seems reasonable to add a > >patch that implements %HQ as a log-format variable to record the HTTP > >query string. Leaving the initial '?' is intentional, but I don't feel > >strongly one way or another. > > How about to use the same function as haproxy? > > http://git.haproxy.org/?p=haproxy-1.5.git;a=blob;f=src/proto_http.c;h=5db64b5bf6b0c02dc8d501087bbb94ec320c2c43;hb=HEAD#l11004 > > find_param_list()
Except that his function performs the lookup into the captured URI, which is available even at the end of the transaction processing. (...) > I would also suggest to check > > #### > if (!txn->uri) { > #### > > before you go further due to the fact that a seach on "<BADREQ>" could > never match ;-) I agree, I found this a bit awkward as well :-) Regards, Willy