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

[expression-language] jayway jsonpath lib has issue for Cache #1713

Closed
yinghe1 opened this Issue Dec 4, 2018 · 2 comments

Comments

@yinghe1
Copy link

yinghe1 commented Dec 4, 2018

Expected Behavior

We intend to use jsonpath function like concat($a, 'sometext') to config gravitee properties. However the underlying lib used jayway jsonpath has a bug json-path/JsonPath#491
Gravitee should probably switch to another jsonpath lib since jayway jsonpath cache has many open issues, make Cache unusable: https://github.com/json-path/JsonPath/issues?utf8=%E2%9C%93&q=cache

Current Behavior

see json-path/JsonPath#491 for details for the problem

Possible Solution

Better switch to a better jsonpath lib.
Temp work around is to use the following before everything loads:
CacheProvider.setCache(new NOOPCache());

Your Environment

  • Version used: gravitee 1.18.x
@brasseld

This comment has been minimized.

Copy link
Member

brasseld commented Dec 4, 2018

Hi @yinghe1

Can you provide a simple API export to reproduce the problem ?

Thanks a lot

@yinghe1

This comment has been minimized.

Copy link
Author

yinghe1 commented Dec 13, 2018

sorry for the late reply. no sample for API export but as json-path/JsonPath#491 shows, if you use any concat("/", $.key) notation to configure properties fields, you will run into this issue

@brasseld brasseld self-assigned this Dec 18, 2018

@brasseld brasseld added the type: bug label Dec 18, 2018

@brasseld brasseld changed the title jayway jsonpath lib has issue for Cache [expression-language] jayway jsonpath lib has issue for Cache Dec 18, 2018

brasseld added a commit to gravitee-io/gravitee-expression-language that referenced this issue Mar 12, 2019

brasseld added a commit to gravitee-io/gravitee-expression-language that referenced this issue Mar 12, 2019

@brasseld brasseld added this to the APIM - 1.24.0 milestone Mar 12, 2019

brasseld added a commit to gravitee-io/gravitee-expression-language that referenced this issue Mar 12, 2019

NicolasGeraud added a commit to gravitee-io/gravitee-expression-language that referenced this issue Mar 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.