-
Couldn't load subscription status.
- Fork 31
Add field exclusion from queryables endpoint. #489
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
Conversation
|
Hey @jonhealy1 could you please take a look? |
| - Excluded fields will not appear in the queryables response | ||
| - Excluded fields and their nested children will be skipped during field traversal | ||
| - Both the field itself and any nested properties will be excluded | ||
|
|
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.
Can you move this section to the main Readme and add a link to the section in the Table of Contents?
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.
This looks really good. Can you add a test?
CHANGELOG.md
Outdated
|
|
||
| - CloudFerro logo to sponsors and supporters list [#485](https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/pull/485) | ||
| - Latest news section to README [#485](https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/pull/485) | ||
| - Environment variable `EXCLUDED_FROM_QUERYABLES` to exclude specific fields from queryables endpoint and filtering. Supports comma-separated list of fully qualified field names (e.g., `properties.auth:schemes,properties.storage:schemes`) |
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.
Here if you can add the link with the pr number
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.
I fixed it when i fixed the conflict in the changelog
|
|
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.
@MathewNWSH Nice work thanks!
Description:
added
EXCLUDED_FROM_QUERYABLESenvironment variable to allow excluding specific fields from the queryables endpoint and filtering, even if those are indexed. Supports comma-separated list of fully qualified field names (e.g., properties.auth:schemes,properties.storage:schemes).PR Checklist:
pre-commit run --all-files)make test)