Skip to content

ManagedIdentityClient(..., client_capabilities=["cp1"]).acquire_token_for_client(..., claims_challenge="...") #791

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

Merged
merged 1 commit into from
Apr 3, 2025

Conversation

rayluo
Copy link
Collaborator

@rayluo rayluo commented Feb 12, 2025

This pull request adds the access_token_sha256_to_refresh parameter to the acquire_token_for_client method in the msal.ConfidentialClientApplication class.

This PR meets feature requirements and acceptance criteria.

This PR supports ManagedIdentityClient(..., client_capabilities=["cp1"]).acquire_token_for_client(..., claims_challenge="...").

Currently, the new behavior will be available for Service Fabric. Other Managed Identity v1 providers may pick up those new behaviors soon. After that happens, we will enable this behavior for more Managed Identity code paths.

@rayluo rayluo force-pushed the at-to-refresh branch 3 times, most recently from 2e6dc04 to f05ae3c Compare March 2, 2025 00:13
@rayluo rayluo force-pushed the at-to-refresh branch 2 times, most recently from 92ae1e3 to 05b1424 Compare March 19, 2025 23:14
@rayluo rayluo changed the title ConfidentialClientApplication(...).acquire_token_for_client(..., access_token_sha256_to_refresh=...) ManagedIdentityClient(..., client_capabilities=["cp1"]).acquire_token_for_client(..., claims_challenge="...") Mar 19, 2025
@rayluo rayluo marked this pull request as ready for review March 19, 2025 23:25
@rayluo rayluo requested a review from a team as a code owner March 19, 2025 23:25
@rayluo rayluo merged commit 2f9747f into dev Apr 3, 2025
18 checks passed
@rayluo rayluo deleted the at-to-refresh branch April 3, 2025 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants