Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Capture requirements for JSONPath #100

Closed
mmccool opened this issue Nov 18, 2020 · 4 comments
Closed

Capture requirements for JSONPath #100

mmccool opened this issue Nov 18, 2020 · 4 comments

Comments

@mmccool
Copy link
Contributor

mmccool commented Nov 18, 2020

IETF has a group attempting to standardize JSON path, we should capture our use case and requirements (eg security...) and provide our input.

Edit: see https://datatracker.ietf.org/group/jsonpath/about/

@farshidtz
Copy link
Member

farshidtz commented Nov 23, 2020

Is it the same jsonpath group mentioned in #56 (comment)?

I think operators for ISO 8601 timestamp comparison are necessary. String operators are not very handy and often require complex regular expressions (related to #80).

Geospatial queries are nice to have, in case they are part of a TD (related to #79).

OData examples from OGC SensorThings:

  • Time comparison: resultTime ge 2010-06-01T00:00:00Z
  • Geo distance: geo.distance(Locations/location, geography’POINT(-122, 43)’) gt 1

@mmccool
Copy link
Contributor Author

mmccool commented Nov 23, 2020

Notes:

  • Yes, as mentioned in issue 56 so I added a link to the IETF group to the description
  • a joint meeting with SDW/OGC is planned for Dec 10, we can discuss what should be supported there; this is closely tied to how geospatial data is represented in TDs or directory metadata about TDs (and there is still the issue of "dynamic" geolocation data reported via a property...)

@mmccool
Copy link
Contributor Author

mmccool commented Nov 30, 2020

Generally I have a concern that arbitrary JS should not be allowed in queries for security reasons (including DoS attacks by specifying pathologically expensive queries). Instead there should be a specific set of comparison operators and query types supported. I hope this aligns with what the IETF group is thinking.

@farshidtz
Copy link
Member

From Discovery call:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants