bhollis/jsonview

Error parsing escaped double quotes

Closed this issue · 8 comments

de-cix-msauer commented

Hi,

somehow it now fails to parse properly escaped double quotes in string values.

{ "value":[ { "@some.text":"W/\"12241774\"" } ] }

expected ',' or '}' after property value in object at line 4 column 26

Never had this issues before. Seems to be new in 1.2.1.

jgpacker commented

Came here to report the same thing.

{"name":"Smartphone Motorola Moto G4 PLUS XT1640 - Dual Chip,Tela 5.5\",Câmera 16MP+Frontal 5MP,Octa Core,32GB,2GB RAM,Android 6,Sensor impressão digital"}
bhollis commented
bhollis commented
  • 👍1
bhollis commented

I've submitted a fix to addons.mozilla.org. That regex was more trouble than it was worth, so I just replaced it with some simple code that's guaranteed to be correct. You should see an updated addon in a day or so once Mozilla has reviewed it.

  • 👍1
cscorley commented

I think I have a similar case (perhaps #142?), where I receive this error:

There was an error parsing the JSON document. The document may not be well-formed.
expected ',' or '}' after property value in object at line 2 column 22

A minimal case:

{
"dots":"\"v4.0\" \"v4.0\""
}

If I recall correctly, prior to 1.2.3 & 1.2.2, the error would occur at the first period. In 1.2.3, the error occurs at the second.

I do recall that a month or so ago, this string was not causing a problem.

bhollis commented

Thanks, I'll push out another fix for that. This is what I get for not putting my full attention on this.

bhollis commented

That fix has been released.

cscorley commented

@bhollis Excellent! Thank you for your work.