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
