-
Notifications
You must be signed in to change notification settings - Fork 292
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature | Provide the same Managed Identity support as Azure.Identity #947
Comments
Hi @MarcWils Yes, we will be working on that soon, now that we've bumped up minimum .NET Framework version supported to be v4.6.1. We will be replacing calls for token acquisition to use Azure.Identity instead and the same way for Managed Identity authentication. |
Thanks @cheenamalhotra. What does "(access_token caching) is out of scope" mean? I'm hoping that is going to be part of this implementation. Thanks!! |
Hi @ericsampson There are 2 access token topics in that thread:
|
Thanks, there's a lot of moving pieces to keep track of in these conversations, especially since they're spread across a bunch of issues :) |
#616 introduced support for Managed Identities.
This may work for the most common scenarios. However, both Azure.Identity and Microsoft.Azure.Services.AppAuthentication have implementations which also support Service Fabric.
Using the SqlAuthenticationProvider from Microsoft.Azure.Services.AppAuthentication works in a Service Fabric application. The one from Microsoft.Data.SqlClient doesn't.
This may also be related to #771. However, the discussion there seems to mainly focussing on caching the access token.
It seems a choice has to be made between extending the built-in support in Microsoft.Data.SqlClient or providing seamless integration with Azure.Identity.
The text was updated successfully, but these errors were encountered: