-
Notifications
You must be signed in to change notification settings - Fork 21
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
CMR-10151: Add Collection Search Endpoint #349
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a test that does the POST request for the collection search?
Also looks like there is a snyk vulnerability that needs to be addressed. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #349 +/- ##
==========================================
- Coverage 88.77% 87.56% -1.22%
==========================================
Files 29 23 -6
Lines 1515 1094 -421
Branches 298 238 -60
==========================================
- Hits 1345 958 -387
+ Misses 161 136 -25
+ Partials 9 0 -9 ☔ View full report in Codecov by Sentry. |
Adding support for collection-search:
Adds collection search extension to conformsTo
Initially, I considered adding a /collections/search route for searching collections. While this is a valid approach, it's more aligned with STAC browser to implement collection search directly at the /collections level. (see discussion here).
A user can now search for collection based on spatial and temporal fields.
In STAC browser there is now a
Search for Collections
tabCollection Search though a
CURL
curl "http://localhost:3000/stac/LPCLOUD/collections?bbox=-180,-90,180,90"
curl -X POST "http://localhost:3000/stac/PROVIDER_ID/collections" \ -H "Content-Type: application/json" \ -d `{ "bbox": [-180, -90, 180, 90], "datetime": "2021-01-01T00:00:00Z/2021-12-31T23:59:59Z", "limit": 10}'