-
Notifications
You must be signed in to change notification settings - Fork 5
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
Keyword search does not work on registry-api deployed on pds.nasa.gov #227
Comments
@jimmie can you confirm you are seeing this same issue on a local deployment? |
This break requirement #460 |
yes, I am seeing this on a local deployment of the registry service from the latest code on the main branch, running against the production EN Opensearch. |
Hi @jjacob7734 , @jimmie , actually the bug is in the documentation where Upgrade is done in commit c3fe6f21da8ea87fe58b6dcf9f250d34aa85db43 |
Thanks, @tloubrieu-jpl. There's one more small update needed in the documentation at https://nasa-pds.github.io/pds-api/guides/search/endpoints.html. I see you updated to |
🐛 Describe the bug
The result from a search query to https://pds.nasa.gov/api/search/1/ does not change if the
keyword
modifier is used. The documentation at https://nasa-pds.github.io/pds-api/guides/search/endpoints.html indicates thatkeyword
is a valid parameter. The example given iskeyword=insight
.📜 To Reproduce
Steps to reproduce the behavior:
curl --get 'https://pds.nasa.gov/api/search/1/products' --header 'Accept: application/json' | json_pp > out_nokeyword.json
keyword=insight
specified:curl --get 'https://pds.nasa.gov/api/search/1/products?keyword=insight' --header 'Accept: application/json' | json_pp > out_keyword_insight.json
out_nokeyword.json
andout_keyword_insight.json
to see that they are the same.🕵️ Expected behavior
I expected the
keyword=insight
modifier to limit the result to only those products that match theinsight
keyword.📚 Version of Software Used
This uses the search API deployed at https://pds.nasa.gov/api/search/1/.
🩺 Test Data / Additional context
🏞Screenshots
🖥 System Info
🦄 Related requirements
⚙️ Engineering Details
The text was updated successfully, but these errors were encountered: