parser: logfmt: bare keys get true as value, not nil (fluent#6099)

seveas · root · commit 6114162da2d0 · 2023-02-08T11:42:10.000Z
In fluent#4359, @nokute78 notes that bare keys in logfmt lines get the wrong value after parsing: they should get a `true` value, not `nil`. Signed-off-by: Dennis Kaarsemaker <dennis@kaarsemaker.net> Signed-off-by: root <root@sumit-acs.novalocal>
diff --git a/src/flb_parser_logfmt.c b/src/flb_parser_logfmt.c
@@ -182,7 +182,7 @@ static int logfmt_parser(struct flb_parser *parser,
                                 msgpack_pack_str(tmp_pck, 0);
                             }
                             else {
-                                msgpack_pack_nil(tmp_pck);
+                                msgpack_pack_true(tmp_pck);
                             }
                         }
                         else {

Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ static int logfmt_parser(struct flb_parser *parser,`
`182`	`182`	`msgpack_pack_str(tmp_pck, 0);`
`183`	`183`	`}`
`184`	`184`	`else {`
`185`		`- msgpack_pack_nil(tmp_pck);`
	`185`	`+ msgpack_pack_true(tmp_pck);`
`186`	`186`	`}`
`187`	`187`	`}`
`188`	`188`	`else {`