Skip to content

Commit 04cf99a

Browse files
committed
fixed PR comments
1 parent 3c9b653 commit 04cf99a

File tree

5 files changed

+15
-14
lines changed

5 files changed

+15
-14
lines changed

articles/communication-services/concepts/identity-model.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,8 +167,8 @@ This architecture eliminates the need for a separate identity management service
167167

168168
### Limitations
169169
The Microsoft Entra ID integration is currently in public preview and has the following limitations:
170-
- [Continuous Access Evaluation](/entra/identity/conditional-access/concept-continuous-access-evaluation) is not available. To revoke access tokens immediately, follow the instructions in [Create and manage access tokens for end users](../quickstarts/identity/access-tokens.md#create-and-manage-access-tokens-for-end-users).
171-
- Removing an Entra ID user does not automatically remove all associated data from the Communication Services resource. To ensure all data is deleted, follow the instructions in [Create and manage access tokens for end users](../quickstarts/identity/access-tokens.md#create-and-manage-access-tokens-for-end-users).
170+
- [Continuous Access Evaluation](/entra/identity/conditional-access/concept-continuous-access-evaluation) is not available. To revoke access tokens immediately, follow the instructions in [Revoke access tokens](../quickstarts/identity/access-tokens.md?pivots=platform-azcli#revoke-access-tokens).
171+
- Removing an Entra ID user does not automatically remove all associated data from the Communication Services resource. To ensure all data is deleted, follow the instructions in [Delete an identity](../quickstarts/identity/access-tokens.md?pivots=platform-azcli#delete-an-identity).
172172

173173
## Next steps
174174

articles/communication-services/quickstarts/identity/entra-id-authentication-integration.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ ms.custom: mode-other, devx-track-js, has-azure-ad-ps-ref
1818

1919
[!INCLUDE [Public Preview Disclaimer](../../includes/public-preview-include.md)]
2020

21-
This quickstart demonstrates how to use the Communication Services Common SDK along with Azure Identity SDK in a console application to authenticate a Microsoft Entra ID user, obtain an Azure Communication Services access token for Microsoft Entra ID user. The resulting Azure Communication Services access token allows you to integrate calling and chat features using the Communication Services Calling and Chat SDKs.
21+
This quickstart demonstrates how to use the Communication Services Common SDK along with Azure Identity SDK in a console application to authenticate a Microsoft Entra ID user and obtain an Azure Communication Services access token. The resulting Azure Communication Services access token allows you to integrate calling and chat features using the Communication Services Calling and Chat SDKs.
2222

2323
## Prerequisites
2424
- An Azure account with an active subscription. [Create an account for free](https://azure.microsoft.com/free/?WT.mc_id=A261C142F).
@@ -76,7 +76,7 @@ Users must be authenticated against Microsoft Entra applications with Azure Comm
7676

7777
The following application settings influence the experience:
7878
- The *Supported account types* property defines whether the application is single tenant ("Accounts in this organizational directory only") or multitenant ("Accounts in any organizational directory"). For this scenario, you can use multitenant.
79-
- *Redirect URI* defines the URI where the authentication request is redirected after authentication. For this scenario, you can use **Public client/native (mobile & desktop)** and enter **`http://localhost`** as the URI.
79+
- *Redirect URI* defines the URI where the authentication request is redirected after authentication.
8080

8181
For more detailed information, see [Register an application with the Microsoft identity platform](/entra/identity-platform/quickstart-register-app#register-an-application).
8282

@@ -112,6 +112,7 @@ In the Azure portal follow these steps:
112112
7. In the **Client ID** field, enter the client ID of Contoso application from [step 2](entra-id-authentication-integration.md#step-2-create-a-microsoft-entra-application-registration-or-select-a-microsoft-entra-application).
113113
8. Click **Save and exit** to apply the changes.
114114

115+
![Providing Entra ID users with access to the Azure Communication Services resource through the Azure portal](./media/entra-id/acs-resource-access-managing.png)
115116

116117
### Step 5: Provide Administrator consent and group access to Azure Communication Services Clients application
117118

articles/communication-services/quickstarts/identity/includes/entra-id/support-entra-id-users-js.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ ms.author: aigerimb
1515

1616
## Set up prerequisites
1717

18-
- [Node.js](https://nodejs.org/) Active LTS and Maintenance LTS versions (8.11.1 and 10.14.1 recommended).
19-
- [Azure Identify SDK for JavaScript](https://www.npmjs.com/package/@azure/identity) to authenticate with Microsoft Entra ID.
18+
- [Node.js](https://nodejs.org/)
19+
- [Azure Identity SDK for JavaScript](https://www.npmjs.com/package/@azure/identity) to authenticate with Microsoft Entra ID.
2020
- [Azure Communication Services Common SDK for JavaScript](https://www.npmjs.com/package/@azure/communication-common) to obtain Azure Communication Services access tokens for Microsoft Entra ID user.
2121

2222
## Final code
@@ -43,24 +43,24 @@ npm init -y
4343
Use the `npm install` command to install the Azure Identity and Azure Communication Services Common SDKs for JavaScript.The Azure Communication Services Common SDK version should be `2.4.0` or later.
4444

4545
```console
46-
npm install @azure/communication-common@latest --save
47-
npm install --save @azure/identity
46+
npm install @azure/communication-common --save
47+
npm install @azure/identity --save
4848
npm install express --save
4949
npm install dotenv --save
5050
```
5151

5252
The `--save` option lists the library as a dependency in your **package.json** file.
5353

54-
## Set up the app framework
54+
## Implement the credential flow
5555

5656
From the project directory:
5757

5858
1. Open a new text file in your code editor
59-
1. Create the structure for the program, including basic exception handling and importing following SDK classes:
59+
1. Create a basic NodeJS Express server and add the following imports:
6060

6161
```javascript
6262
const { AzureCommunicationTokenCredential } = require('@azure/communication-common');
63-
const {InteractiveBrowserCredential} = require('@azure/identity');
63+
const { InteractiveBrowserCredential } = require('@azure/identity');
6464
const express = require("express");
6565

6666
// You will need to set environment variables in .env
@@ -125,7 +125,7 @@ app.get('/', async (req, res) => {
125125
}
126126
});
127127
```
128-
Providing scopes is optional. When not specified, the `https://communication.azure.com/clients/.default` scope is automatically used, requesting all API permissions for Communication Services Clients.
128+
Providing scopes is optional. When not specified, the `https://communication.azure.com/clients/.default` scope is automatically used, requesting all API permissions for Communication Services Clients that have been registered on the client application.
129129

130130
<a name='step-3-obtain-acs-access-token-of-the-entra-id-user'></a>
131131

articles/communication-services/quickstarts/identity/includes/entra-id/support-entra-id-users-net.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ dotnet add package Azure.Identity
4848
dotnet add package Azure.Communication.Common
4949
```
5050

51-
### Set up the app framework
51+
### Implement the credential flow
5252

5353
From the project directory:
5454

@@ -117,7 +117,7 @@ var credential = new CommunicationTokenCredential(entraTokenCredentialOptions);
117117

118118
```
119119

120-
Providing scopes is optional. When not specified, the `https://communication.azure.com/clients/.default` scope is automatically used, requesting all API permissions for Communication Services Clients.
120+
Providing scopes is optional. When not specified, the `https://communication.azure.com/clients/.default` scope is automatically used, requesting all API permissions for Communication Services Clients that have been registered on the client application.
121121

122122
<a name='step-3-obtain-acs-access-token-of-the-entra-id-user'></a>
123123

506 KB
Loading

0 commit comments

Comments
 (0)