servicebinding / spec

Specification for binding services to k8s workloads

https://servicebinding.io

Apache License 2.0

92 stars 35 forks source link

Define JSON Pointer alternative syntax #189

Closed scothis closed 2 years ago

scothis commented 2 years ago

183 uses JSON Pointer to discover fixed paths within a resource. The syntax `/foo/bar` clashes with JSON Path style syntax `.foo["bar"]`. We notionally want to align on the scope of JSON Pointer with the syntax of JSON Path.

Refs #177

scothis commented 2 years ago

While I do not want to define the grammar in the semantics of a specific implementation. I think we can likely define a subset of allowable nodes https://pkg.go.dev/k8s.io/client-go/util/jsonpath#NodeType to achieve the desired behavior.

Allowed:

List
Field

Needs more thought:

Array
Identifier

Disallowed:

Text
Filter
Int
Float
Wildcard
Recursive
Union
Bool

AST FTW

scothis commented 2 years ago

Folded a proposal into #183

scothis commented 2 years ago

Resolved in #183

servicebinding / spec

Define JSON Pointer alternative syntax #189

183 uses JSON Pointer to discover fixed paths within a resource. The syntax /foo/bar clashes with JSON Path style syntax .foo["bar"]. We notionally want to align on the scope of JSON Pointer with the syntax of JSON Path.

183 uses JSON Pointer to discover fixed paths within a resource. The syntax `/foo/bar` clashes with JSON Path style syntax `.foo["bar"]`. We notionally want to align on the scope of JSON Pointer with the syntax of JSON Path.