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: Silent Flow and Logout #1208
Authorization Code Flow for Single Page Applications: Silent Flow and Logout #1208
Conversation
…n-code-flow-silentHandler
…n-code-flow-silentHandler
General comment: seems like these could be separate PRs, and theres seemingly other unrelated scope creep (e.g. validation in redirect methods). |
…n-code-flow-silentHandler
export type SystemOptions = { | ||
loggerOptions?: BrowserLoggerOptions; | ||
export type BrowserSystemOptions = SystemOptions & { | ||
loggerOptions?: LoggerOptions; |
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.
nitpick: Overwrites a generic loggerOptions object? If it is browser specific, the name is very generic. I will check the code as a whole later, this is for traction.
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.
Logger isn't browser specific, only the implementation is, so I reverted back to generic logger options.
…silentHandler Authorization Code Flow for Single Page Applications: Silent Flow and Logout
This PR adds the silent flow and logout to the library. The implementation of ADAL SSO and other SSO features is included in a later PR.
This PR also enables CSP in our sample applications. This improves the security of our samples, and should be included in documentation moving forward.