-
Notifications
You must be signed in to change notification settings - Fork 305
Description
Issue
Users need to perform manual changes to get template and samples code working in Azure US Gov (a National Cloud). These changes are minimal but obtrusive as we don't describe when a user would normally perform these changes (which is reliant on whether the user is using Public Azure or a National Cloud).
This also adds friction in that the samples in botbuilder-samples aren't as easily deployable as deploying to Public Azure (no code changes required).
Proposed change
Clean up BotFrameworkAdapter by having it check/default for BotOpenIdMetadata and ChannelService environment variables in the constructor/initializer in JS and Python.
These variables are configured in our deployment settings on Azure.
Component Impact
- BotFrameworkAdapter (JS, Python)
- AuthenticationConstants (JS, Python)
- These are namespaced as Constants in
botframework-connectorand not exposed to the developer. In C# these constants are public.
- These are namespaced as Constants in
Tracking Status
Dotnet SDK
PR: .netcore, needs update/fix
Javascript SDK
PR: microsoft/botbuilder-js#975