Typo was introduced in 57bc891 ("BUG/MEDIUM: log: fix risk of segfault when logging HTTP fields in TCP mode") which inverted the condition in the test and caused <BADREQ> to be logged when using %HP.
Signed-off-by: Nenad Merdanovic <nmer...@anine.io> --- src/log.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/log.c b/src/log.c index dffef54..2d02247 100644 --- a/src/log.c +++ b/src/log.c @@ -1917,7 +1917,7 @@ int build_logline(struct stream *s, char *dst, size_t maxsize, struct list *list while (spc < end && *spc != '?' && !HTTP_IS_SPHT(*spc)) spc++; - if (!txn || txn->uri || nspaces == 0) { + if (!txn || !txn->uri || nspaces == 0) { chunk.str = "<BADREQ>"; chunk.len = strlen("<BADREQ>"); } else { -- 2.7.0