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] Modified Troubleshooting guide for CSS #20223

Merged
merged 84 commits into from
Mar 1, 2022

Conversation

KarishmaGhiya
Copy link
Member

Packages impacted by this PR

@azure/identity

Issues associated with this PR

#20068

Describe the problem that is addressed by this PR

Developer Customer Support needs an Azure Identity Trouble Shooting Guide (TSG) for JS which includes the error codes and mitigation explained clearly. .NET has a PR open to address this Azure/azure-sdk-for-net#26716
This PR for JS is based off of .NET's PR and customized for JS Identity SDK

  • Added a changelog (if necessary)
  • Will add the changelog if we decide to ship it part of current preview

### Credential unavailable error

#### Environment variables not configured
> CAUTION: Requests and responses in the Azure Identity library contain sensitive information. Precaution must be taken to protect logs when customizing the output to avoid compromising account security.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point! 🙂

##### ClientCertificate credential
| Error |Description| Mitigation |
|---|---|---|
|`CredentialUnavailableError` thrown with message. "DefaultAzureCredential failed to retrieve a token from the included credentials."|All credentials in the `DefaultAzureCredential` chain failed to retrieve a token, each throwing a `CredentialUnavailableError` themselves|<ul><li>[Enable logging](#enabling-and-configuring-logging) to verify the credentials being tried, and get further diagnostic information.</li><li>Consult the troubleshooting guide for underlying credential types for more information.</li><ul><li>[EnvironmentCredential](#troubleshoot-environment-credential-authentication-issues)</li><li>[ManagedIdentityCredential](#troubleshoot-managed-identity-authentication-issues)</li><li>[VisualStudioCodeCredential](#troubleshoot-visual-studio-code-authentication-issues)</li><li>[VisualStudioCredential](#troubleshoot-visualstudio-credential-authentication-issues)</li><li>[AzureCLICredential](#troubleshoot-azure-cli-authentication-issues)</li><li>[AzurePowershellCredential](#troubleshoot-azure-powershell-authentication-issues)</li></ul>|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here it says thrown with message. “Defau[…] That dot before the quote is weird. Also, can we use the code quote where the message is? So that the message can look as if it was copied as is into the document?

CredentialUnavailableError thrown with message: DefaultAzureCredential failed to retrieve a token from the included credentials.

What do you think?

Comment on lines 314 to 316
|Az.Account module >= 2.2.0 isn’t installed.|The `Az.Account` module needed for authentication in Azure PowerShell isn't installed.|Install the latest `Az.Account` module. Installation instructions can be found [here](https://docs.microsoft.com/powershell/azure/install-az-ps). To use the Azure PowerShell credential, install the Azure Az PowerShell module:```powershell
Install-Module -Name Az -Scope CurrentUser -Repository PSGallery -Force
```
```|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This

```powershell
Install-Module -Name Az -Scope CurrentUser -Repository PSGallery -Force 
```

Is broken. I think the fix is to just inline it, like Install-Module -Name Az -Scope CurrentUser -Repository PSGallery -Force

Copy link
Contributor

@sadasant sadasant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it!

@KarishmaGhiya KarishmaGhiya merged commit 7d1ef56 into Azure:main Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Developer Customer Support needs an Azure Identity Trouble Shooting Guide
3 participants