Skip to content

Conversation

@YuriZmytrakov
Copy link
Collaborator

@YuriZmytrakov YuriZmytrakov commented Sep 8, 2025

Description:

This PR adds a environment variable ENV_MAX_LIMIT, which allows overriding the default MAX_LIMIT value, which controls the ?limit URL parameter used in STAC API queries, defining the maximum number of items and collections that can be returned in a single response.

PR Checklist:

  • Code is formatted and linted (run pre-commit run --all-files)
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable
  • Changes are added to the changelog

@YuriZmytrakov YuriZmytrakov marked this pull request as draft September 8, 2025 09:48
@YuriZmytrakov YuriZmytrakov force-pushed the CAT-1310-2 branch 6 times, most recently from 77e7379 to 80e05d5 Compare September 9, 2025 09:07
@YuriZmytrakov YuriZmytrakov marked this pull request as ready for review September 9, 2025 09:11
Adds env var ENV_MAX_LIMIT, which allows overriding the MAX_LIMIT value,
which controls the ?limit URL parameter used in STAC API queries.
jonhealy1
jonhealy1 previously approved these changes Sep 12, 2025
Copy link
Collaborator

@jonhealy1 jonhealy1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@YuriZmytrakov Nice work

@jonhealy1 jonhealy1 merged commit dd6335a into stac-utils:main Sep 13, 2025
15 checks passed
YuriZmytrakov added a commit to YuriZmytrakov/stac-fastapi-elasticsearch-opensearch that referenced this pull request Sep 15, 2025
**Description:**

This PR adds a environment variable `ENV_MAX_LIMIT`, which allows
overriding the default `MAX_LIMIT` value, which controls the `?limit`
URL parameter used in STAC API queries, defining the maximum number of
items and collections that can be returned in a single response.

**PR Checklist:**

- [x] Code is formatted and linted (run `pre-commit run --all-files`)
- [x] Tests pass (run `make test`)
- [x] Documentation has been updated to reflect changes, if applicable
- [x] Changes are added to the changelog

Co-authored-by: Yuri Zmytrakov <[email protected]>
Co-authored-by: Jonathan Healy <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants