Skip to content

Support standard Azure Function configuration options for managed identity#433

Merged
sebastianburckhardt merged 3 commits intomainfrom
pr/support-af-mi
Jan 14, 2025
Merged

Support standard Azure Function configuration options for managed identity#433
sebastianburckhardt merged 3 commits intomainfrom
pr/support-af-mi

Conversation

@sebastianburckhardt
Copy link
Member

@sebastianburckhardt sebastianburckhardt commented Oct 22, 2024

This PR adds support for configuring Managed Identity using the same configuration-section based approach that is used for other Azure Functions extensions. This makes it simpler to configure MI for .NET apps, and possible to configure it for all AF apps.

For example, Azure Storage can be configured by defining AzureWebJobsStorage__accountName, AzureWebJobsStorage__credential, and AzureWebJobsStorage__clientId, and the Event Hubs namespace can be configured by defining EventHubsConnection__fullyQualifiedNamespace, EventHubsConnection__credential, and EventHubsConnection__clientId.

To avoid dealing with legacy credential types, this PR also replaces most remaining uses of the Microsoft.Azure.Storage.* (V11) SDKs with the Azure.Storage.* (V12) equivalents. Since this is not a major version bump, it does not remove the actual package dependency. Also, a few utility functions (name validation and connection string parsing) still depend on V11 since there is no equivalent in V12.

Remaining work:

  • run basic smoke tests for local and cloud deployment
  • need documentation and perhaps a sample
  • need to test runtime scaling

…ration section, and replace most Azure Storage SDK V11 references with the V12 equivalents.
@sebastianburckhardt
Copy link
Member Author

Since we are doing a major version release for 3.0.0, I have spent a bit more time on this PR in order to remove all dependencies on the old Microsoft.Azure.Storage.Blob.

This required copying some utility code for validation and for connection string parsing which is no longer present in the new storage SDK.

@sebastianburckhardt sebastianburckhardt merged commit d7e3dab into main Jan 14, 2025
@sebastianburckhardt
Copy link
Member Author

remaining work items are tracked in #441

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request P1 Priority 1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants