Skip to content

feat: add optional allow-list for fetching json schemas - #1828

Draft
jsignell wants to merge 1 commit into
stac-utils:mainfrom
jsignell:allow-list
Draft

jsignell wants to merge 1 commit into
stac-utils:mainfrom
jsignell:allow-list

Conversation

@jsignell

@jsignell jsignell commented Oct 6, 2026

Copy link
Copy Markdown
Member

Related Issue(s):

Description:

This PR adds an allow_list attribute on the Validator class. Not sure if this is worth putting into pystac itself or whether we should just show the pattern. Another option could be to provide a new Validator class that has the allow-list populated with some sane schema defaults. So that people could just do:

set_validator(SecureJsonSchemaSTACValidator)

as opposed to:

set_validator(
    JsonSchemaSTACValidator(
        allow_list=[
            "https://schemas.stacspec.org/",
            "https://stac-extensions.github.io/",
        ]
    )
)

PR Checklist:

  • Pre-commit hooks pass (run pre-commit run --all-files)
  • Tests pass (run pytest)
  • Documentation has been updated to reflect changes, if applicable
  • This PR maintains or improves overall codebase code coverage
  • This PR's title is formatted per conventional commits

@jsignell jsignell changed the title Add optional allow-list for fetching json schemas feat: add optional allow-list for fetching json schemas Oct 6, 2026

This branch has not been deployed

No deployments
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.

1 participant