Skip to content

feat: STACCube authentication via GDAL bearer token#61

Merged
luukvdmeer merged 1 commit into
ZGIS:mainfrom
mattyfromtheblock:auth-GDAL-token
Mar 14, 2025
Merged

feat: STACCube authentication via GDAL bearer token#61
luukvdmeer merged 1 commit into
ZGIS:mainfrom
mattyfromtheblock:auth-GDAL-token

Conversation

@mattyfromtheblock

Copy link
Copy Markdown
Contributor

Description

This PR implements token based authentication for the STACCube via GDAL env (passed through stackstac and rasterio)

Relates to # (issue)
n/a

Type of change

Select one or more relevant options:

  • Bug fix (change which fixes a bug reported in a specific issue)
  • New feature (change which adds a feature requested in a specific issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (improved documentation for existing functionality in the package)
  • Repository update (change related to non-package files or structures in the GitHub repository)

Checklist:

  • I have read and understood the contributor guidelines of this project
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I ran all demo notebooks locally and there where no errors

@mattyfromtheblock

mattyfromtheblock commented Mar 10, 2025

Copy link
Copy Markdown
Contributor Author

NOTE: One change in this PR is also covered by #60

The change in question concerns the deprecated "np.NaN" alias, which has been changed to "np.nan" (semantique already exclusively uses "np.nan" from what I've seen. So, the change should not cause any issues)

empty_arr = xr.full_like(extent, np.NaN)

@luukvdmeer luukvdmeer merged commit 79c4687 into ZGIS:main Mar 14, 2025
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