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.

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

Possible to support credentials from azure cli in DefaultAzureCredential? #8566

Closed
damienpontifex opened this issue Nov 10, 2019 · 12 comments
Closed
Assignees
Labels
Azure.Identity Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. feature-request This issue requires a new behavior in the product in order be resolved.
Milestone

Comments

@damienpontifex
Copy link

I noticed in SharedTokenCacheCredentials it only works on windows.

I used to use get_azure_cli_credentials and does the cli have a shared token cache across platforms?

It'd be great for DefaultAzureCredential to work using the logged in azure cli account when developing locally and then no code changes needed when deploying to Azure with managed identity.

@lmazuel lmazuel added Azure.Core Client This issue points to a problem in the data-plane of the library. labels Nov 11, 2019
@lmazuel
Copy link
Member

lmazuel commented Nov 11, 2019

Hi @damienpontifex
We definitely wants to address this scenario, we'll discuss with the team how to do it correctly :)

@mayurid mayurid added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Nov 11, 2019
@damienpontifex
Copy link
Author

Thanks @lmazuel I'd be happy to contribute a change if there's guidance on the best way to do this.

@lmazuel
Copy link
Member

lmazuel commented Nov 18, 2019

Thanks @damienpontifex , we are working on the design , I don't think this issue is the best one for contribution, but I will update this thread as soon as we have a candidate PR.

@jongio
Copy link
Member

jongio commented Dec 4, 2019

FYI - We are still discussing this internally and should have an update soon.

@carlosotgz
Copy link

@jongio Hi. Is there any update for this issue? :)

@jongio
Copy link
Member

jongio commented Jan 29, 2020

Thanks for checking in. We are discussing when to do a preview of this. Hopefully within the next couple of weeks, but we still have some remaining design to flesh out. I'll earmark this to update when we have something ready.

@schaabs

@carlosotgz
Copy link

Hi :) Any luck with this issue? It would be really nice to be able to use play with the SDK without having to create additional service accounts :(

@chlowell
Copy link
Member

The first preview of DefaultAzureCredential integrating the Azure CLI is available now in azure-identity 1.4.0b1. In this preview, DefaultAzureCredential will attempt to authenticate with the identity you've set up with az login, with no configuration required in your code. Please try it out and let us know what you think.

@damienpontifex
Copy link
Author

Thanks @chlowell I did some initial testing with some KeyVault code I have and seems good so far :)

Any idea when these credentials will also make their way to the python azure management libraries? e.g. DNS

@chlowell
Copy link
Member

Adopting these credentials in management libraries is tracked by #9310. I don't know the timeline (@zikalino may have more details) but a comment on that issue links to an adapter you can try in the meantime.

@jongio
Copy link
Member

jongio commented Apr 22, 2020

@damienpontifex - Just so you know the python version of AzureIdentityCredentialWrapper has been added to Azure Identity Extensions https://github.com/jongio/azidext#python

We are still a ways from officially supporting Azure Identity in Mgmt plane libraries, but it is something we are actively working on. For now, we must use adapters.

@chlowell chlowell added the feature-request This issue requires a new behavior in the product in order be resolved. label Apr 27, 2020
@chlowell chlowell self-assigned this Apr 27, 2020
@chlowell chlowell added this to the [2020] June milestone Apr 27, 2020
@chlowell
Copy link
Member

Thanks for opening this issue. I'm closing it now because the current preview version of azure-identity has this support. Please open another issue if you encounter problems using it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Azure.Identity Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. feature-request This issue requires a new behavior in the product in order be resolved.
Projects
None yet
Development

No branches or pull requests

6 participants