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 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 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 ceciliaavila added the Automation: Parity with js The PR needs to be ported to JS label May 23, 2023
@ceciliaavila 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 merged commit 48084a6 into main May 23, 2023
@tracyboehrer tracyboehrer deleted the southworks/add/auth-blobstorage-using-managedidentity branch May 23, 2023 19:57
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