curveball / browser

A HAL browser middleware for node.js
MIT License
44 stars 5 forks source link

JSON viewer should add double-quotes around properties that need it. #166

Open evert opened 1 year ago

evert commented 1 year ago

We're basically using JSON5 syntax, and while this is legal:

{
  foo: "bar"
}

If the key contains (for example) a space we should encode like this:

{
  "foo 1 2": "bar"
}

So we should add quotes if the key name would be illegal with the quotes. This also includes empty keys, e.g:

{
  "": "hello world, yes this is legal
}

Real world example:

image