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

2387: JSONPath eval(string,string) issue fix #2388

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

swapdewalkar
Copy link

@swapdewalkar swapdewalkar commented Apr 1, 2024

What this PR does / why we need it?

Address the issue:
alibaba/fastjson#4493
#2387

Summary of your change

If you are passing string as root object to eval its fail as current eval(Object ..) method can not handle it.
Add new method and using extract method to handle this case.
Also added the test case

Please indicate you've done the following:

  • Made sure tests are passing and test coverage is added if needed.
  • Made sure commit message follow the rule of Conventional Commits specification.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

@CLAassistant
Copy link

CLAassistant commented Apr 1, 2024

CLA assistant check
All committers have signed the CLA.

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

Successfully merging this pull request may close these issues.

2 participants