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

[Bug] Azure ML MSI support is missing #7489

Open
bgavrilMS opened this issue Jan 2, 2025 · 1 comment · May be fixed by #7512
Open

[Bug] Azure ML MSI support is missing #7489

bgavrilMS opened this issue Jan 2, 2025 · 1 comment · May be fixed by #7512
Assignees
@Robbie-Microsoft
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

Azure ML Studio uses similar environment variables to other hosting environments such as Azure App Service. In our previous implementation, we attempted to detect Azure App Service's environment variables before CloudShell. Because CloudShell uses a subset of the environment variables, it will be wrongly detected if it is attempted first in an AzureML environment and fails to add the value of the MSI_SECRET env var in a required header named secret.

See MSAL Python's implementation: https://github.com/AzureAD/microsoft-authentication-library-for-python/blob/dev/msal/managed_identity.py#L415

Original issue:

AzureAD/microsoft-authentication-library-for-dotnet#4984
@bgavrilMS bgavrilMS added bug A problem that needs to be fixed for the feature to function as intended. confidential-client Issues regarding ConfidentialClientApplications labels Jan 2, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 2, 2025

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 Jan 2, 2025
@github-actions github-actions bot closed this as completed Jan 2, 2025
@bgavrilMS bgavrilMS reopened this Jan 2, 2025
@bgavrilMS bgavrilMS added p2 P1 and P2 are priorities of the bug. P2 bugs should get fixed/closed within 3 months. and removed more-information-needed Use this label when you are waiting on information from the issue creator labels Jan 2, 2025
@Robbie-Microsoft Robbie-Microsoft linked a pull request Jan 15, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Robbie-Microsoft Robbie-Microsoft

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
2 participants
@bgavrilMS @Robbie-Microsoft