Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
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

#6636
Merged
tracyboehrer merged 1 commit into
mainmicrosoft/botbuilder-dotnet:mainfrom
southworks/add/auth-blobstorage-using-managedidentitymicrosoft/botbuilder-dotnet:southworks/add/auth-blobstorage-using-managedidentityCopy head branch name to clipboard
May 23, 2023
Merged

[#6346] Add Azure Storage (BlobStorage.cs) using a managed identity#6636
tracyboehrer merged 1 commit into
mainmicrosoft/botbuilder-dotnet:mainfrom
southworks/add/auth-blobstorage-using-managedidentitymicrosoft/botbuilder-dotnet:southworks/add/auth-blobstorage-using-managedidentityCopy head branch name to clipboard

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
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.