Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

[#6346] Add Azure Storage (BlobStorage.cs) using a managed identity - #6636

Merged
tracyboehrer (tracyboehrer) merged 1 commit into
mainfrom
southworks/add/auth-blobstorage-using-managedidentity
May 23, 2023
Merged

[#6346] Add Azure Storage (BlobStorage.cs) using a managed identity#6636
tracyboehrer (tracyboehrer) merged 1 commit into
mainfrom
southworks/add/auth-blobstorage-using-managedidentity

Conversation

@ceciliaavila

Copy link
Copy Markdown
Collaborator

Fixes #6346

Description

This PR enables MSI authentication for Azure Blob Storage.

Specific Changes

  • Added a new constructor with the necessary parameters for creating the blob storage client using a tokenCredential.

Testing

The following image shows a bot saving data in the blob storage after using the tokenCredential to authenticate.
image

The following image shows the new unit test passing.
image

@ceciliaavila Cecilia Avila (ceciliaavila) added the Automation: Parity with js The PR needs to be ported to JS label May 23, 2023
@ceciliaavila
Cecilia Avila (ceciliaavila) requested a review from a team as a code owner May 23, 2023 13:48
@coveralls

Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 352848

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 2 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.003%) to 78.98%

Files with Coverage Reduction New Missed Lines %
/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs 1 90.91%
/libraries/AdaptiveExpressions/BuiltinFunctions/GetPreviousViableTime.cs 1 90.91%
Totals Coverage Status
Change from base Build 351887: -0.003%
Covered Lines: 25914
Relevant Lines: 32811

💛 - Coveralls

@tracyboehrer
tracyboehrer (tracyboehrer) deleted the southworks/add/auth-blobstorage-using-managedidentity branch May 23, 2023 19:57
This was referenced May 14, 2026
This was referenced May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Automation: Parity with js The PR needs to be ported to JS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Azure Storage (BlobStorage.cs) using a managed identity

4 participants