-
Notifications
You must be signed in to change notification settings - Fork 2k
Description
Per the Breaking Changes section of the changelog for 0.50.0 linking to this PR, and as discussed in #23036 (comment), this is a known breaking change, albeit one that presents a blocker for many Vector users to taking recent versions of Vector.
As such, I'm creating this issue to document the concern and as a basis for putting up a PR to re-add support for Azure token auth via environment variables.
A note for the community
- Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
- If you are interested in working on this issue or have submitted a pull request, please leave a comment
Problem
#23351 bumped the version of several Azure Rust SDKs to unblock various other features. However, due to the apparent abandonment of the azure_storage/_blobs crates, it was not possible at the time to bump those crates to the same version as azure_core and azure_identity. This, combined with the removal of EnvironmentCredential from recent versions of azure_identity, made it extremely difficult for Vector to continue supporting Azure token auth in general, and via environment variables in particular.
However, as there are obviously a number of other users of the Azure Rust SDK in the wild, there is prior art for re-adding support for token auth (see eg this comment referencing a PR in Metaswitch). This, combined with copying in EnvironmentCredential from older versions of azure_identity should make it possible to restore the missing functionality.
I'll be putting up a PR soon which does exactly that.
Configuration
Version
0.50.0
Debug Output
Example Data
No response
Additional Context
No response
References
No response