We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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 Framework Composer integration with Power Virtual Agents v1 is broken on .test and .preprod environments. Cannot download the .zip. Looks related to island urls or something. I'm still investigating, but getAuthCredentials: BotFramework-Composer/Composer/packages/server/src/externalContentProvider/powerVirtualAgentsProvider.ts at bc5cc73817881d4835f7c8ef68a6d6be905f88ee · microsoft/BotFramework-Composer appears to be wrong. We are always using prod credentials, even when opening a test or ppe bot.
if (url.hostname.includes('.int.') || url.hostname.includes('.ppe.')) { log('Using INT / PPE auth credentials.'); return { clientId: COMPOSER_1P_APP_ID, scopes: [${PVA_TEST_APP_ID}/.default], targetResource: PVA_TEST_APP_ID, };
${PVA_TEST_APP_ID}/.default
Likely the hostname check needs to include .test. but currently only checks for .int. and .ppe.
latest
Steps to reproduce the behavior:
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Describe the bug
Bot Framework Composer integration with Power Virtual Agents v1 is broken on .test and .preprod environments. Cannot download the .zip. Looks related to island urls or something. I'm still investigating, but getAuthCredentials: BotFramework-Composer/Composer/packages/server/src/externalContentProvider/powerVirtualAgentsProvider.ts at bc5cc73817881d4835f7c8ef68a6d6be905f88ee · microsoft/BotFramework-Composer appears to be wrong. We are always using prod credentials, even when opening a test or ppe bot.
if (url.hostname.includes('.int.') || url.hostname.includes('.ppe.')) {
log('Using INT / PPE auth credentials.');
return {
clientId: COMPOSER_1P_APP_ID,
scopes: [
${PVA_TEST_APP_ID}/.default
],targetResource: PVA_TEST_APP_ID,
};
Likely the hostname check needs to include .test. but currently only checks for .int. and .ppe.
Version
latest
Browser
OS
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Screenshots
Additional context
The text was updated successfully, but these errors were encountered: