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

paging and next links #155

Closed
matthewhanson opened this issue Feb 13, 2019 · 1 comment
Closed

paging and next links #155

matthewhanson opened this issue Feb 13, 2019 · 1 comment
Milestone

Comments

@matthewhanson
Copy link
Member

sat-api uses the parameters page and limit when the number of Items is more than limit.

This is not compliant with STAC because STAC doesn't specify a mechanism to do paging. The assumption is that all results will be returned, although this can cause errors with sat-api when there are more than around 920 items (sat-utils/sat-search#69)

See the STAC issue discussing this:
radiantearth/stac-spec#381

Currently the next links are formed by convert the POST parameters to a GET request. However there are two issues:

cc @vincentsarago @sharkinsspatial

@matthewhanson
Copy link
Member Author

The page parameter is compliant starting with STAC 0.7.0, so this part is fine. However, the next link still needs to stringify the JSON.

matthewhanson added a commit that referenced this issue Jun 3, 2019
stringify objects for next link, resolves #155
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

No branches or pull requests

1 participant