Hi Stéphane, On Wed, May 17, 2017 at 02:03:26AM +0200, Stéphane Cottin wrote: > Thanks, this is the right answer, recompile with DEFINE=-DREQURI_LEN=4096 > fixes this issue.
that's what I was going to say :-) > This limit should be raised at runtime according to the log "len" option, if > you want long log lines, you want them complete :) In fact no, but we should have a global tunable for it now. The reason why we recommend to always keep the URI last is to ensure logs can safely be truncated at the least interesting place, because for most users, the long trail of query string arguments are mostly useless, while for captures, states and timers there is no compromise. If you have the time to provide a patch doing this, that's clearly something we could merge. It's not too hard, REQURI_LEN is used at only a few places. You could have global.requri_len which would be set using tune.http.log_uri_len for example. Best regards, Willy
