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

Update docs for DefaultAzureCredential regarding AZURE_CLIENT_ID / managedIdentityClientId relationship #13462

Closed
christothes opened this issue Jan 28, 2021 · 3 comments · Fixed by #13548
Closed

Update docs for DefaultAzureCredential regarding AZURE_CLIENT_ID / managedIdentityClientId relationship #13462

christothes opened this issue Jan 28, 2021 · 3 comments · Fixed by #13548
Assignees
@sadasant
Labels
Azure.Identity Client This issue points to a problem in the data-plane of the library. Docs
Milestone
[2021] February

Comments

@christothes
Copy link
Member

Add documentation to DefaultAzureCredential indicating that setting the AZURE_CLIENT_ID environment variable is equivalent to setting the managedIdentityClientId.
@ramya-rao-a ramya-rao-a added the Client This issue points to a problem in the data-plane of the library. label Jan 29, 2021
@ramya-rao-a ramya-rao-a added this to the [2021] February milestone Jan 29, 2021
@ramya-rao-a
Copy link
Contributor

@christothes Can you provide some context here? What documentation are we referring to? What confusion lead us to make this clarification?

@christothes
Copy link
Member Author

christothes commented Jan 29, 2021 •
edited
Loading

Sure - This issue was the catalyst for the issue.

Relevant documentation is here:

BTW, I created one of these for each language.

@sadasant
Copy link
Contributor

I think I might be able to do this this month. Worst case scenario I move it to next month.

christothes added a commit that referenced this issue Feb 2, 2021
@christothes
Update Identity readme
39f26d4 
fixes ##13462
@christothes christothes mentioned this issue Feb 2, 2021
Merged
@ghost ghost closed this as completed in #13548 Feb 3, 2021
ghost pushed a commit that referenced this issue Feb 3, 2021
@christothes
Update Identity readme (#13548)
a97f98f 
fixes #13462
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sadasant sadasant

Labels
Azure.Identity Client This issue points to a problem in the data-plane of the library. Docs
Projects
None yet
Milestone
[2021] February
Development

Successfully merging a pull request may close this issue.

Update Identity readme Azure/azure-sdk-for-js
3 participants
@sadasant @christothes @ramya-rao-a