-
Notifications
You must be signed in to change notification settings - Fork 191
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
Managed Identity for Azure VM, App Service, Service Fabric, etc. #480
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8bb07b0
to
33f1157
Compare
e2b5aa8
to
302d2fd
Compare
jmprieur
reviewed
Nov 18, 2022
ef3c9d4
to
1e4426b
Compare
jiasli
reviewed
Mar 16, 2023
jiasli
reviewed
Mar 16, 2023
jiasli
reviewed
Mar 16, 2023
Cloud Shell is just one type of managed identity. Perhaps we should follow the same interface for Cloud Shell? |
jiasli
reviewed
Apr 7, 2023
84d68be
to
696c787
Compare
xiangyan99
approved these changes
May 10, 2024
ec8b018
to
9350391
Compare
jiasli
reviewed
Jun 17, 2024
jiasli
reviewed
Jun 17, 2024
jiasli
suggested changes
Jun 17, 2024
jiasli
reviewed
Jun 17, 2024
jiasli
reviewed
Jun 21, 2024
jiasli
reviewed
Jun 21, 2024
jiasli
reviewed
Jun 21, 2024
e3e0412
to
c336d2d
Compare
Fix docs Adjusting error message and docs Fix typo
Error out on platforms other than Linux and Windows
Congratulations on MSAL's support for managed identity. I am sure our customers will benefit from this great new feature! 🎉 |
[heart] Yong Zhang reacted to your message:
…________________________________
From: Jiashuo Li ***@***.***>
Sent: Monday, June 24, 2024 12:09:02 PM
To: AzureAD/microsoft-authentication-library-for-python ***@***.***>
Cc: Subscribed ***@***.***>
Subject: Re: [AzureAD/microsoft-authentication-library-for-python] Managed Identity for Azure VM, App Service, Service Fabric, etc. (PR #480)
Congratulations on MSAL's support for managed identity. I am sure our customers will benefit from this great new feature! 🎉
—
Reply to this email directly, view it on GitHub<#480 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AM4LM2CVUJ2OZR5RHJFHZBDZJAD55AVCNFSM5XVXKEA2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJYGY2DENRSHE4Q>.
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Note: This is a proof-of-concept, which means there is no guarantee that this behavior will be eventually included into MSAL Python.
There are two new APIs added.
The high level API works for your confidential client which federated with a managed identity.This will be moved into a separated PR for its own consideration.The low level API acquires token for managed identity
More details of the new APIs are available here.
In order to test this PR on Azure VM, you would need to:
pip install --force-reinstall "git+https://github.com/AzureAD/microsoft-authentication-library-for-python.git@mi"
To test this on App Service
To test this on Azure Functions
To test this on Azure Automation (we have not tested this)
msal
package. But it seems Azure Automation only supports installing a package with its dependencies from PyPI. This PR is not currently available from PyPI, so, we are unable to test this.To test this on Service Fabric
test steps in Azure SDK
Note:
https://github.com/AzureAD/microsoft-authentication-library-for-python/pull/420
.Once merged, this PR will resolve #548. Also, it will officially close #487 as the callback is no longer needed.