Skip to content
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.

Sign up for GitHub

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

Jump to bottom

AcquireTokenForManagedIdentity should throw if a user-assigned managed identity clientID or resourceID is supplied when not supported #7198

Closed
bgavrilMS opened this issue Jul 12, 2024 · 2 comments
Closed

AcquireTokenForManagedIdentity should throw if a user-assigned managed identity clientID or resourceID is supplied when not supported #7198

bgavrilMS opened this issue Jul 12, 2024 · 2 comments
Labels
bug A problem that needs to be fixed for the feature to function as intended. confidential-client Issues regarding ConfidentialClientApplications p2 P1 and P2 are priorities of the bug. P2 bugs should get fixed/closed within 3 months.

Comments

@bgavrilMS
Copy link
Member

The following ManagedIdentitySources do not currently support user-assigned managed identities:

AzureArc, CloudShell, ServiceFabric

If a user-assigned clientId or resourceId is specified for these sources, we should throw with a message similar to:

https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/blob/5c7c527b173b5f2a719926fb89ccd68bc55d1b3e/src/client/Microsoft.Identity.Client/MsalErrorMessage.cs#L424

This behavior should be consistent cross-language.

Original issue:

AzureAD/microsoft-authentication-library-for-dotnet#4837
@bgavrilMS bgavrilMS added bug A problem that needs to be fixed for the feature to function as intended. p2 P1 and P2 are priorities of the bug. P2 bugs should get fixed/closed within 3 months. confidential-client Issues regarding ConfidentialClientApplications labels Jul 12, 2024
@github-actions GitHub Actions
Copy link
Contributor

Invalid Issue Template:
Please open a new issue and use one of the provided issue templates. Thanks!

@github-actions github-actions bot added the more-information-needed Use this label when you are waiting on information from the issue creator label Jul 12, 2024
@bgavrilMS bgavrilMS reopened this Jul 13, 2024
@bgavrilMS bgavrilMS removed the more-information-needed Use this label when you are waiting on information from the issue creator label Jul 13, 2024
@rayluo
Copy link

rayluo commented Jul 16, 2024

Discussed here. Feel free to reopen this issue if a discrepancy will be observed.

@rayluo rayluo closed this as completed Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug A problem that needs to be fixed for the feature to function as intended. confidential-client Issues regarding ConfidentialClientApplications p2 P1 and P2 are priorities of the bug. P2 bugs should get fixed/closed within 3 months.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rayluo @bgavrilMS