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

C# nlp-with-dispatch calls for wrong key #1076

Closed
jwiley84 opened this issue Dec 11, 2018 · 1 comment
Closed

C# nlp-with-dispatch calls for wrong key #1076

jwiley84 opened this issue Dec 11, 2018 · 1 comment
Assignees
Labels
P1 Painful if we don't fix, won't block releasing

Comments

@jwiley84
Copy link
Contributor

csharp_dotnetcore #14 nlp-with-dispatch bot throws an error if the dispatch service in the .bot file doesn't have a subscriptionKey. Except dispatch services don't have subscription keys, only authoring keys. This error needs to be changed to ask for authoringKey instead.

dispatch error

@stevengum
Copy link
Member

The DispatchService can contain a SubscriptionKey. When the Dispatch entry in the bot file is created, only the AuthoringKey is required, not the optional SubscriptionKey. This can lead to no "subscriptionKey" entry being added to the .bot file.

msbot-connect-dispatch-4 3 2-134

The Node.js sample uses the authoringKey for the DispatchService and the LuisServices, so the samples need to be brought into alignment.

@sgellock, I'm thinking that we should follow the Node.js sample's lead and have the bot use the authoring key, but add a comment indicating that's only for prototyping.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 Painful if we don't fix, won't block releasing
Projects
None yet
Development

No branches or pull requests

4 participants