-
Notifications
You must be signed in to change notification settings - Fork 486
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
Bot is not accepting v2 tokens from Bot Framework Emulator - Single Tenant Bots #6877
Comments
Thank you for your inquiry about your Teams app development issue! |
A few questions:
|
Hi @dmvtech I'm currently using the latest SDK code and the latest version of the Emulator. The issue is reproducible with the local endpoint. I've tried selecting the "Use version 1.0 authentication tokens" option in the Emulator, but the token received by AD is still a v2 token. This is because we are modifying the app registration manifest to share v2 tokens. The problem lies in this code botbuilder-dotnet/libraries/Microsoft.Bot.Connector/Authentication/EmulatorValidation.cs Line 82 in e449b3c
In a Single Tenant authentication scenario, when the Emulator sends a request with v2 tokens, we are not considering the endpoint https://login.microsoftonline.com/{tenant-id}/v2.0 as a valid issuer. However, in a Multi Tenant authentication scenario, we have this endpoint added as a valid issuer: botbuilder-dotnet/libraries/Microsoft.Bot.Connector/Authentication/EmulatorValidation.cs Line 31 in e449b3c
|
Was able to reproduce. Needed to restart Emulator entirely after making the manifest change to |
Version
latest
Describe the bug
Bot fails with the error below when bot is called from emulator by passing Microsoft App ID, Microsoft App password, Tenant ID
To Reproduce
Steps to reproduce the behavior:
appsetting.json
of the botExpected behavior
Bot should allow v2 tokens.
The following code needs to be updated to allow v2 tokens when calls come from Emulator.
botbuilder-dotnet/libraries/Microsoft.Bot.Connector/Authentication/EmulatorValidation.cs
Line 82 in e449b3c
The text was updated successfully, but these errors were encountered: