BotFramework-Emulator icon indicating copy to clipboard operation
BotFramework-Emulator copied to clipboard

fix: [#2398] Support Bots that use Single Tenant AAD Applications

Open ceciliaavila opened this issue 1 year ago • 1 comments

Fixes #2398

Description

This PR enables the tenant id value to authenticate Emulator with Single Tenant apps.

This feature requires bots with the last updates of botbuilder-dotnet and botbuilder-js: PR#6769 PR#4643

Specific Changes

  • Added tenantId property to the classes involved in authentication.
  • Added logic to get the access token for the Single Tenant login endpoint.
  • Added method to use tenant ID value to create a dynamic issuer.

Testing

These images show the new field Tenant ID in the Open Bot and New Bot Configuration views. image image

This is the new message error related to authentication. image

The following image shows the Emulator working with a bot using Single Tenant authentication. image

ceciliaavila avatar Apr 05 '24 14:04 ceciliaavila

Coverage Status

coverage: 67.547% (-0.004%) from 67.551% when pulling c804d815a6256c664dcbbd447a73c1155a4ef7c3 on southworks/add/support-singletenant-apps into 012a6641077fe96c8155687f8e6ecb1ebffdc0e3 on main.

coveralls avatar Apr 05 '24 14:04 coveralls

@tracyboehrer is there anywhere I can get a build of the emulator with this PR in it? The nightlies repo doesn't have any recent builds. I could build it myself, looking to avoid the time spent getting all of the build tools stood up so I can focus on testing a bot. I'm hoping this exists and I just missed it.

Thanks!

GaryIrick avatar May 30 '24 17:05 GaryIrick

@GaryIrick At the moment there isn't. This also relies on a change on the SDK side to take the tenant into account when validating the token. That has just now made it to the DotNet SDK.

tracyboehrer avatar May 30 '24 17:05 tracyboehrer