Json parser gets tripped up on curly brace in property value

[springmeyer]

This valid json fails to parse (when cache_features:False):

{
  "type": "FeatureCollection",
  "features": [
    {
      "type": "Feature",
      "properties": {
        "curly1": "}"
      },
      "geometry": {
        "type": "Point",
        "coordinates": [ -122,48 ]
      }
    }
  ]
}

So does this fail to parse:

{
  "type": "FeatureCollection",
  "features": [
    {
      "type": "Feature",
      "properties": {
        "curly1": "{"
      },
      "geometry": {
        "type": "Point",
        "coordinates": [ -122,48 ]
      }
    }
  ]
}

However, curiously, this does parse:

{
  "type": "FeatureCollection",
  "features": [
    {
      "type": "Feature",
      "properties": {
        "curly1": "}",
        "curly2": "{"
      },
      "geometry": {
        "type": "Point",
        "coordinates": [ -122,48 ]
      }
    }
  ]
}

[artemp]

@springmeyer - yep, bug, working on fix.

Background : curly brackets are used to determine feature boundary in GeoJSON when 'cache-features="false"(extract_bounding_box grammar). Currently we process every{ }` even if they are a part of a quoted string:

feature = raw[lit('{')[_a = 1]
                  >> *(eps(_a > 0) >> (lit('{')[_a += 1]
                                       |
                                       lit('}')[_a -=1]
                                       |
                                       coords[_b = _1]                                      
                                       |
                                       char_))][push_box(_r1, _r2, _b, _1)]

[artemp]

@springmeyer - this is now fixed in 6325bf2

[springmeyer]

✔️ - confirmed fix locally. Want to add regression test before closing?

[springmeyer]

tests added in 16e6db3, and fixed up (to actually test both modes of cache_features in b54b275