-
Notifications
You must be signed in to change notification settings - Fork 479
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
SSO Protocol Support #3360
SSO Protocol Support #3360
Conversation
…#3244) * Added code to initialize security context during ContinueConversationAsync. * Addressed comments from carlos. * More comments. * Wrapped the code to add trustServiceUrl in a skills check to ensure it is only done when the request has been handled by the skill infrastructure.
…to prevent the ASP.Net runtime from registering it as a controller.
…ollerAbstract Made ChannelServiceController abstract
…3294) * Enabled Deserialization for OAuthPrompt Dialog With TypeHandling.None * Return default(T) incase of null object * Account for obj type equal to JArray
…event deserialization exceptions
CR feedback + tests
resolved credential provider issue.
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.6.3 |
} | ||
else | ||
{ | ||
throw new InvalidOperationException($"ExchangeAsyncAsync returned improper result: {result.GetType()}"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling: ExchangeAsync
fixes #3356
Added schema updates, token service API calls, and OAuthPrompt updates to support SSO (single sign on) protocol operations for user login flows.
@ted and @gabog , can you make sure your skill OAuth calls are still right?