Skip to content
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

endpointKey is populated with the subscriptionId value in .bot file using Azure login #1286

Closed
stevkan opened this issue Feb 1, 2019 · 2 comments · Fixed by #1301
Closed
Assignees
Labels
Approved Approved by the Triage team Bug Your classic code defect P2 Nice to have

Comments

@stevkan
Copy link
Collaborator

stevkan commented Feb 1, 2019

Version

4.2.1

Describe the bug

When adding QnA as a service via Azure login the endpointKey is populated with the subscriptionId value. Subsequently, the actual endpointKey key:value is not being stored in the .bot file.

To Reproduce

Steps to reproduce the behavior:

  1. Open your .bot file in Emulator
  2. Select 'Add QnA Maker' under Services and login via Azure
  3. Select a knowledge base and add it
  4. Observe added JSON in .bot file

Expected behavior

Both the endpointKey and subscriptionId keys should have the appropriate values

{
  "type": "qna",
  "endpointKey": "subscriptionId",
  "hostname": "hostname",
  "id": "id",
  "kbId": "kbId",
  "name": "name"
}

[bug]

@cwhitten cwhitten added Approved Approved by the Triage team Bug Your classic code defect labels Feb 4, 2019
@cwhitten cwhitten added P2 Nice to have 4.3 labels Feb 4, 2019
@a-b-r-o-w-n
Copy link
Contributor

@stevkan I have not been able to reproduce this issue following the steps provided. The value I see for endpointKey corresponds to a key in my cognitive services azure resource.

@a-b-r-o-w-n
Copy link
Contributor

@stevkan I see it now. I was unaware of the endpoint key on the QnA side. Let me dig into this a little more.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Approved by the Triage team Bug Your classic code defect P2 Nice to have
Projects
None yet
4 participants