-
Notifications
You must be signed in to change notification settings - Fork 14
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
pagination/next link #83
Comments
Included |
While I see the next relation type being included, it's not clear how standards should behave in non-GET requests. Doesn't seem useful to let this open. Then each extension will define their own behavior. For reference, here's one "extension" defining it: https://github.com/radiantearth/stac-api-spec/blob/master/api-spec.md#paging-extension |
@m-mohr The |
@ghobona The support for the |
5/10/21 SWG - OK to close. |
Features API requires a next link (rel=
next
) on search results, which seems like it belongs in Commons, especially if #82 is done.An issue with next link is what happens if the query was a POST request, because converting that to a GET request could be problematic for large queries. One way would be for the implementor to use sessions and return a token to get the next page of results, but we didn't want that to be required.
Instead we return a next link that also has fields for the body and headers so that the user can use those to make the request again, along with the parameters returned in the next link. It's not very elegant by any stretch.
Regardless, I think Commons should include the next link, and can discuss what to do in the POST vs GET case.
The text was updated successfully, but these errors were encountered: