json2dict converts unquoted true and false to numerical boolean

[ramikhaldi]

The json2dict treats the unquoted string values true and false as boolean and converts them to 1 and 0.
Example:
{
"valueTrue": true,
"valueFalse": false
}

will be converted to:

valueTrue 1 valueFalse 0

Is there any configuration to prevent that ?

Thanks in advance!

[bovine]

That's an interesting observation. In general, we've tried to make the yajl::json2dict function perform identically to the same one provided by the json package, except faster. However, this is indeed an unintended difference in behavior:

package require json

::json::json2dict {{
"valueTrue": true,
"valueFalse": false
}}

valueTrue true valueFalse false

[bovine]

I think changing the replacements in parse2dict_boolean_callback might fix this behavior? https://github.com/flightaware/yajl-tcl/blob/master/generic/yajltcl.c#L227

[ramikhaldi]

Thanks @bovine for your response, I also have seen that location but I was not sure if there might be other side effects when fixing it. As there are already tests for the library, we can rerun the tests and see if something has been broken. I will recompile the library and check.

[ramikhaldi]

@bovine I have performed a pull request. Let's see the test results in the ci..

[ramikhaldi]

Can you please approve the waiting workflows (#39)? Thanks.

[bovine]

That PR has been merged and released as v1.8.1