This pull request has been inactive for 60 days. If the pull request is still relevant please comment to re-activate the pull request. If no action is taken within 7 days, the pull request will be marked closed.

@rd-danny-fleer this removes the possibility to use json path.

The code should be fixed by correcting the inverted logic :

_, err := jp.ParseString(claim)
if err == nil {
         return src.Get(claim).Interface()

And not

	_, err := jp.ParseString(claim)
if err != nil {
          return src.Get(claim).Interface()

Testing for $. prefix could detect the expression is a JSONPath.

@rd-danny-fleer this removes the possibility to use json path.

The code should be fixed by correcting the inverted logic :

_, err := jp.ParseString(claim)
if err == nil {
return src.Get(claim).Interface()

And not

	_, err := jp.ParseString(claim)

if err != nil {
return src.Get(claim).Interface()

Testing for $. prefix could detect the expression is a JSONPath.

Would you be willing to open a PR for this change and add more extensive tests?

Will do.

I'd prefer JMESPath but JSONPATH would be good enough in most cases.