Skip to content

Commit 34eba1e

Browse files
authored
[Identity] Cleanups (Azure#34236)
- Remove `ApplicationCredential` from the Identity codebase. This has been released in `2.0.0-beta.5` and removed in `2.0.0-beta.6`. It is never GA and there is no plan for this. - Remove test suites under `manual-integration` that have been automated under `integration` - Remove mentions of `UsernamePasswordCredential` and related information in the Troubleshooting guide
1 parent e9e7974 commit 34eba1e

38 files changed

+5
-1572
lines changed

sdk/identity/identity/TROUBLESHOOTING.md

Lines changed: 5 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ This troubleshooting guide covers the following areas of the Azure Identity clie
2020
- [Permission issues](#permission-issues)
2121
- [Troubleshoot default Azure credential authentication issues](#troubleshoot-default-azure-credential-authentication-issues)
2222
- [Troubleshoot environment credential authentication issues](#troubleshoot-environment-credential-authentication-issues)
23-
- [Troubleshoot username and password authentication issues](#troubleshoot-username-and-password-authentication-issues)
2423
- [Troubleshoot service principal authentication issues](#troubleshoot-service-principal-authentication-issues)
2524
- [Troubleshoot managed identity authentication issues](#troubleshoot-managed-identity-authentication-issues)
2625
- [Azure Virtual Machine managed identity](#azure-virtual-machine-managed-identity)
@@ -210,36 +209,12 @@ The `DefaultAzureCredential` attempts to retrieve an access token by sequentiall
210209

211210
### Client authentication error
212211

213-
The `EnvironmentCredential` supports service principal authentication and username and password authentication.
214-
Follow the troubleshooting guidelines below for the respective authentication type that failed.
212+
The `EnvironmentCredential` supports service principal authentication.
213+
Follow the troubleshooting guidelines below for the authentication type that failed.
215214

216-
| Authentication Type | Troubleshooting Guide |
217-
| ---------------------------------------------- | ----------------------------------------------------------------------------------------- |
218-
| ClientSecret/ClientCertificate/ClientAssertion | [Service principal auth guide](#troubleshoot-service-principal-authentication-issues) |
219-
| Username and password | [Username and password auth guide](#troubleshoot-username-password-authentication-issues) |
220-
221-
## Troubleshoot username and password authentication issues
222-
223-
### AuthenticationRequiredError
224-
225-
| Error Code | Issue | Mitigation |
226-
| ----------- | -------------------------------------------- | ----------------------------------------------------------------------------------------- |
227-
| AADSTS50126 | The provided username or password is invalid | Ensure the `username` and `password` provided when constructing the credential are valid. |
228-
229-
### Two-factor authentication required error
230-
231-
The `UsernamePasswordCredential` works only for users whose two-factor authentication has been disabled in Microsoft Entra ID. You can change the multi-factor authentication in the Azure portal with the steps [here](https://learn.microsoft.com/entra/identity/authentication/howto-mfa-userstates#change-the-status-for-a-user).
232-
233-
### Request body must contain the following parameter: 'client_assertion' or 'client_secret'
234-
235-
The error `The request body must contain the following parameter: 'client_assertion' or 'client_secret'`, occurs because of how the Microsoft Entra app is configured. The Microsoft Entra app registration seems to be configured as a confidential app. The `UsernamePasswordCredential` works only with public clients and doesn't support confidential apps. To support confidential apps, use either `ClientSecretCredential` or `ClientCertificateCredential` instead.
236-
237-
To allow public client authentication on your Microsoft Entra tenant:
238-
239-
1. In the Azure portal, navigate to the **Authentication** page.
240-
2. Scroll to the bottom of the page. You'll see something that says **Allow public client flows**. Near that, you'll see a **yes** / **no** toggle. Set this toggle to **yes**.
241-
242-
After that, you shouldn't need to specify a client secret to authenticate with this credential.
215+
| Authentication Type | Troubleshooting Guide |
216+
| ---------------------------------------------- | ------------------------------------------------------------------------------------- |
217+
| ClientSecret/ClientCertificate/ClientAssertion | [Service principal auth guide](#troubleshoot-service-principal-authentication-issues) |
243218

244219
## Troubleshoot service principal authentication issues
245220

sdk/identity/identity/src/credentials/azureApplicationCredential-browser.mts

Lines changed: 0 additions & 41 deletions
This file was deleted.

sdk/identity/identity/src/credentials/azureApplicationCredential.ts

Lines changed: 0 additions & 38 deletions
This file was deleted.

sdk/identity/identity/src/credentials/azureApplicationCredentialOptions.ts

Lines changed: 0 additions & 18 deletions
This file was deleted.

sdk/identity/identity/test/internal/node/azureApplicationCredential.spec.ts

Lines changed: 0 additions & 67 deletions
This file was deleted.

sdk/identity/identity/test/manual-integration/AzureFunctions/INSTRUCTIONS.md

Lines changed: 0 additions & 135 deletions
This file was deleted.

sdk/identity/identity/test/manual-integration/AzureFunctions/IdentityTest/.funcignore

Lines changed: 0 additions & 6 deletions
This file was deleted.

sdk/identity/identity/test/manual-integration/AzureFunctions/IdentityTest/.gitignore

Lines changed: 0 additions & 4 deletions
This file was deleted.

sdk/identity/identity/test/manual-integration/AzureFunctions/IdentityTest/.vscode/extensions.json

Lines changed: 0 additions & 3 deletions
This file was deleted.

sdk/identity/identity/test/manual-integration/AzureFunctions/IdentityTest/KeyvaultAuthentication/function.json

Lines changed: 0 additions & 17 deletions
This file was deleted.

0 commit comments

Comments
 (0)