Skip to content

Fixes 944 | Remove the check for returning more than one JSON result in Response Plugin #5192

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

Merged

Conversation

bbbco
Copy link
Contributor

@bbbco bbbco commented Sep 16, 2022

changelog(Fixes): Fixed an issue where the Response template wouldn't return more than one result when parsing a JSON response by path

Fixes #944

Removes the check for returning more than one JSON result in the Response Plugin. If the result is an array of objects, the code following the removed check will just stringify the result. This allows the response to be more easily chained and parsed by the JSONPath plugin.

bbbco added 2 commits October 25, 2022 09:55
…an array of objects, the following lines will just stringify. This allows the response to be parsed by JSONPath
@filfreire filfreire force-pushed the plugin-response_remove-check-for-multiple-results branch from 67eba76 to 34d6a6e Compare October 25, 2022 08:55
@filfreire
Copy link
Contributor

filfreire commented Oct 25, 2022

Tested locally, LGTM

image

@filfreire filfreire enabled auto-merge (squash) October 25, 2022 09:07
@filfreire filfreire merged commit 7dd07ba into Kong:develop Oct 25, 2022
@bbbco
Copy link
Contributor Author

bbbco commented Oct 25, 2022

Thanks for pulling this in!

@bbbco bbbco deleted the plugin-response_remove-check-for-multiple-results branch October 25, 2022 14:09
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.

[Feature Request] Allow multiple results for JSONPath in JSON bodies
2 participants