-
Notifications
You must be signed in to change notification settings - Fork 48
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
Cannot query non-SSL catalogs #88
Comments
Looking at stac-utils/stac-api-validator#34, isn't this an issue with franklin's SSL configuration? I'm not sure passing through a custom I tried to stand up a local franklin instance w/ the latest examples in the stac-spec and got some ingestion errors. If there were a publicly-available franklin instance that I could use to reproduce the error that would be helpful; I'd like to explore the motivating issue to learn more about this use case. |
Franklin itself doesn't have an SSL configuration -- that's on the end user. I opened the previous issue to validate a local Franklin running via a docker-compose file.
that's fine -- the custom StacIO could inherit from StacApiIO and override the
The issue doesn't exist on deployed services with SSL. It only exists on non-SSL instances. You can import a small catalog like this https://github.com/jisantuc/purescript-stac/blob/main/.github/workflows/ci.yml#L21-L28 (your compose file might be different or you can just borrow the compose file from |
But it looks like you're making an |
(verified with |
No problem! Glad it's an easy fix! :-D |
The default
Client.open
method uses aStacIO
implementation based onrequests
, which by default gets real mad about servers without SSL. This makes it difficult to use tools that depend onpystac-client
'sClient.open
in CI, e.g. stac-utils/stac-api-validator#34There's a workaround here, which is calling
from_file
directly, but it might be easier ifopen
just took an optionalStacIO
as well and forwarded it to its own call tofrom_file
The text was updated successfully, but these errors were encountered: