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
Authorization Code Flow for Single Page Applications: Response Handling Code #1183
Authorization Code Flow for Single Page Applications: Response Handling Code #1183
Conversation
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.
few comments
…ation-code-flow-handleTokenResponse
…ation-code-flow-handleTokenResponse
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.
A few questions, otherwise looks good.
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.
Approved with comments, can be addressed in later PRs.
…handleTokenResponse Authorization Code Flow for Single Page Applications: Response Handling Code
This PR adds the response handling to the authorization code flow for single page applications.
This PR does NOT cover token caching or cache cleanup, those will be addressed in the next immediate PR. Please review PRs #1164 and #1181 before reviewing this PR.