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

[Identity] UsernamePasswordCredential should expose authenticate method which returns AuthenticationRecord #11546

Closed
jiasli opened this issue May 20, 2020 · 1 comment
Assignees
Labels
Azure.Identity feature-request This issue requires a new behavior in the product in order be resolved.

Comments

@jiasli
Copy link
Member

jiasli commented May 20, 2020

UsernamePasswordCredential should expose authenticate method which returns AuthenticationRecord.

Otherwise when username+password is used as the authentication method, Azure CLI can't persist user's auth record and utilize refresh tokens stored in MSAL cache. In such case, Azure CLI will have to save user's password by itself, which is absolutely not a good idea from a security perspective.

@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label May 20, 2020
@jiasli jiasli added Azure.Identity feature-request This issue requires a new behavior in the product in order be resolved. labels May 20, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label May 20, 2020
@chlowell
Copy link
Member

Duplicate of #11449

@chlowell chlowell marked this as a duplicate of #11449 May 20, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Azure.Identity feature-request This issue requires a new behavior in the product in order be resolved.
Projects
None yet
Development

No branches or pull requests

2 participants