test-fullautomation / python-jsonpreprocessor

A preprocessor for json files
Apache License 2.0
2 stars 2 forks source link

Colon inside string harms parameter substitution #92

Closed HolQue closed 12 months ago

HolQue commented 1 year ago

This works:

"test_parameter" : "ABC : DEF",

But in case of a dollar operator expression is involved also:

"test_parameter" : "ABC ${value} : DEF"

I get the following error message:

Expecting ',' delimiter

But colons inside strings must not harm the JSON parser.

HolQue commented 1 year ago

Retest not successful.

Issue still needs to be fixed.

test-fullautomation commented 1 year ago

removed "fix in review" state.

test-fullautomation commented 1 year ago

Hi @namsonx , what is the status of this ticket? Thank you, Thomas

namsonx commented 1 year ago

Hello Thomas, Hello Holger,

I made the new commit aa18fff1 on stabi branch to fix the issue which is mentioned in this ticket.

Thank you, Son

HolQue commented 1 year ago

Retest successful. Ticket can be closed.