-
Notifications
You must be signed in to change notification settings - Fork 135
Change the order of credentials in the DefaultAzureCredential
#4669
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
Comments
Here are the order that other languages are following:
|
Hi @antkmsft, we deeply appreciate your input into this project. Regrettably, this issue has remained unresolved for over 2 years and inactive for 30 days, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support. |
@scottaddie: "Wow the order of credentials was determined in the C++ DefaultAzureCredential. We typically put the dev tool credentials at the very end of the chain. We see Managed Identity as more important."
@antkmsft: "The order is the one I presented to the C++ team initially, but you are right, we should try to change it. Initially, I think, ManagedIdentityCredential was at the end because unlike other languages, our ManagedIdentityCredential does not probe the IP address yet, and just tries to get the token. But still, that should not be the factor that prevents it from being in the middle of the chain. The good thing is that everywhere in documentation we say that although we are documenting the order of credentials, we may change their order in the future, and that would not be considered a breaking change. So let me to open an issue to change the order, and update the docs and the SVG."
The text was updated successfully, but these errors were encountered: