Skip to content

Comments

feat: add support for other Azure credentials#3482

Merged
snowman2 merged 5 commits intorasterio:mainfrom
jessjaco:azure-session-updates
Dec 27, 2025
Merged

feat: add support for other Azure credentials#3482
snowman2 merged 5 commits intorasterio:mainfrom
jessjaco:azure-session-updates

Conversation

@jessjaco
Copy link
Contributor

This is a solution for #3423, adding support for all env var based authentication methods for reading data for Azure blob storage, as described here. The logic mimics that defined in the corresponding gdal code.

The solution ended up more verbose than anticipated to maintain existing logic. Specifically, if keyword arguments are passed to AzureSession, they always take precedent over existing environmental variables.

@jessjaco jessjaco changed the title feat: added support for other Azure credentials feat: add support for other Azure credentials Dec 13, 2025
@snowman2 snowman2 added this to the 1.5.0 milestone Dec 15, 2025
@snowman2
Copy link
Member

@jessjaco I am planning on making a final release candidate on 12/29. If you would like this change in 1.5 this will need to be merged before then.

@jessjaco
Copy link
Contributor Author

Ok, I altered the changes to make the logic clearer. I also removed the class variables, since they are available in the credentials object.

Copy link
Member

@snowman2 snowman2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work @jessjaco! Thanks 👍

@snowman2 snowman2 merged commit aa98fbc into rasterio:main Dec 27, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants