Skip to content
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

Add fsspec authentication environment variables to deltares example notebook. #301

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

ghidalgo3
Copy link
Contributor

If fsspec​ sees a URI that starts with https://​ (that was previously in the asset href of the deltares STAC items) then fsspec assumes that URI is publicly accessible and doesn’t try to use any of the authentication kwargs you pass to xarray. fsspec will only use the authentication arguments if the URI has a protocol for an fsspec backend, abfs://​ in our case.

But we need to set the environment variables because there is 2 layers of requests happening, the first for the index file (xarray will forwards kwargs correctly for that one) and the second for the chunk ranges (xarray will not forward kwargs correctly for that one, or at least I couldn’t figure it out). The environment variables allow us to “remotely” tell fsspec what credentials to unconditionally use for all requests.

@777arc 777arc self-requested a review December 5, 2024 16:57
@ghidalgo3 ghidalgo3 merged commit 12d330b into main Dec 5, 2024
5 checks passed
chpolste added a commit to CLIMAAX/FLOODS that referenced this pull request Dec 6, 2024
Following the changes in microsoft/PlanetaryComputerExamples#301.
The adlfs package is now required for data access and has been added to the environment.
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