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.

Sign up for GitHub

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

Jump to bottom

Switch from JsonSmartJsonProvider to JacksonJsonProvider as default #513

Open
pmouawad opened this issue Dec 23, 2018 · 2 comments
Open

Switch from JsonSmartJsonProvider to JacksonJsonProvider as default #513

pmouawad opened this issue Dec 23, 2018 · 2 comments

Comments

@pmouawad
Copy link

Hello,
First thanks for your great library and work on it.

Looking at refreshing our dependencies at JMeter, I noticed json-smart was kind of abandoned by its author.

So should'nt you switch to Jackson ?

By the way when do you plan a new release ?

Thank you and happy holidays.
Regards
@aaime
Copy link

aaime commented Jan 24, 2019

I've been successfully using jayway's json-path as a testing tool for our JSON output generation.
I'm now considering it as a runtime dependency, and the depending on json-smart (as said, an abandoned library, by its author admission) is standing out as negative.

I was not aware of JacksonJsonProvider, but it seems that due to #274 the json-smart dependency cannot be considered optional... pity.

I apologize I cannot offer development support to do eventual switches, just voicing another concern about the json-smart dependency, which you might want to consider when evaluating issues priorities.

@cmunilla
Copy link

cmunilla commented Apr 9, 2019

I provided an modular implementation of JsonPath allowing you to use jackson implementation only if you prefer

mkr added a commit to mkr/JsonPath that referenced this issue Apr 13, 2023
@mkr
Enable using JsonPath without json-smart dependency (json-path#608, a…
33b53c0 
…lso related to json-path#682, json-path#676, json-path#513, json-path#252)

* Replace hard-coded json-smart parser with one retrieved from context (where available) or default configuration.
* This enables a project to _exclude_ the transitive json-smart dependency when setting an alternative default configuration.
@kwvanderlinde kwvanderlinde mentioned this issue Jul 5, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@aaime @cmunilla @pmouawad