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

summary-only does not work on products of a collection #152

Closed
Tracked by #27
tloubrieu-jpl opened this issue Jun 21, 2022 · 3 comments
Closed
Tracked by #27

summary-only does not work on products of a collection #152

tloubrieu-jpl opened this issue Jun 21, 2022 · 3 comments
Assignees
Labels
B13.0 bug Something isn't working s.high High severity

Comments

@tloubrieu-jpl
Copy link
Member

tloubrieu-jpl commented Jun 21, 2022

🐛 Describe the bug

📜 To Reproduce

Steps to reproduce the behavior:

  1. Go to http://pds.nasa.gov/api/search-sbnpsi/1.0/collections/:lidvid/products?summary-only=true
  2. See error 500

In addition, the summary.properties field is not populated in the response when the following request is done:

curl --location --request GET 'http://pds.nasa.gov/api/search-sbnpsi/1.0/collections/urn:nasa:pds:orex.ovirs:data_calibrated/products'

🕵️ Expected behavior

There should not be an error but a response containing only the summary of the result.

The list of properties should be filled in the summary object when the request work.

📚 Version of Software Used

1.0.0

🩺 Test Data / Additional context

🏞Screenshots

🖥 System Info

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

🦄 Related requirements

⚙️ Engineering Details

@tloubrieu-jpl
Copy link
Member Author

@gxtchen this should be tested from:

the latest stable version of the registry-api (1.0.2) which can be deployed using the latest stable registry docker compose (1.0.2)
the latest development version should also be tested but the CICD did not work on registry-api

@jimmie
Copy link
Member

jimmie commented Sep 15, 2022

Seems the removal of summary-only in favor of limit=0 has broken this once again.

@jimmie jimmie assigned jimmie and unassigned al-niessner Sep 15, 2022
@jimmie
Copy link
Member

jimmie commented Sep 16, 2022

Note: this bug was applicable to all /class/{id}/class type requests.

PR approved and merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B13.0 bug Something isn't working s.high High severity
Projects
None yet
Development

No branches or pull requests

4 participants