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

Remove msrestazure from python deps #28231

Open
gdippolito opened this issue Jan 22, 2024 · 3 comments
Open

Remove msrestazure from python deps #28231

gdippolito opened this issue Jan 22, 2024 · 3 comments
Assignees
@jiasli
Labels
AAD Auto-Assign Auto assign by bot Azure CLI Team The command of the issue is owned by Azure CLI team customer-reported Issues that are reported by GitHub users external to the Azure organization. Feature Candidate feature-request
Milestone
Backlog

Comments

@gdippolito
Copy link

Related command
N/A

Is your feature request related to a problem? Please describe.

I believe msrestazure has been widely deprecated and should be removed from the python dependencies.

Describe the solution you'd like
I would like msrestazure to not be part of the Python deps for azure-cli-core

Describe alternatives you've considered
None

Additional context

msrestazure will also bring adal(Microsoft Azure Active Directory Authentication Library) as a dependency which is also inactive and deprecated.
@microsoft-github-policy-service microsoft-github-policy-service bot added customer-reported Issues that are reported by GitHub users external to the Azure organization. Auto-Assign Auto assign by bot AAD labels Jan 22, 2024
@yonzhan
Copy link
Collaborator

yonzhan commented Jan 22, 2024

Thank you for opening this issue, we will look into it.

@yonzhan yonzhan added this to the Backlog milestone Jan 22, 2024
@yonzhan yonzhan added Azure CLI Team The command of the issue is owned by Azure CLI team feature-request labels Jan 22, 2024
@gdippolito
Copy link
Author

Is there a target date for this dependency to be removed from azure-cli? The repository has been archived as of the 10th of April 2024 so I think it would be quite important to remove. See here for reference.

@jiasli
Copy link
Member

jiasli commented May 7, 2024

We currently can't remove the dependency on msrestazure as we still use it for managed identity authentication. Migrating managed identity authentication from msrestazure to MSAL is tracked by #25860.

Is there a target date for this dependency to be removed from azure-cli?

We currently don't have a solid ETA for that yet.

@jiasli jiasli mentioned this issue Jul 26, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jiasli jiasli

Labels
AAD Auto-Assign Auto assign by bot Azure CLI Team The command of the issue is owned by Azure CLI team customer-reported Issues that are reported by GitHub users external to the Azure organization. Feature Candidate feature-request
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
3 participants
@jiasli @gdippolito @yonzhan