Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

feat: grant plugins the ability to specify options to the SSO handler

Browse Source 
... to be handled in the plugin itself (overriding the passport prototype's authorizationParams method)

+ new hook filter:auth.options
This commit is contained in:
Julian Lam
2021-01-13 16:57:48 -05:00
parent 9b289eca9d
commit ab11435ed5
1 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/routes/authentication.js
View File

@@ -98,13 +98,21 @@ Auth.reloadRoutes = async function (params) {
loginStrategies = loginStrategies || []; loginStrategies = loginStrategies || [];
loginStrategies.forEach(function (strategy) { loginStrategies.forEach(function (strategy) {
if (strategy.url) { if (strategy.url) {
router.get(strategy.url, Auth.middleware.applyCSRF, function (req, res, next) { router.get(strategy.url, Auth.middleware.applyCSRF, async function (req, res, next) {
req.session.ssoState = req.csrfToken && req.csrfToken(); let opts = {
passport.authenticate(strategy.name, {
scope: strategy.scope, scope: strategy.scope,
prompt: strategy.prompt || undefined, prompt: strategy.prompt || undefined,
state: strategy.checkState ? req.session.ssoState : undefined, };
})(req, res, next);
if (strategy.checkState) {
req.session.ssoState = req.csrfToken && req.csrfToken();
opts.state = req.session.ssoState;
}
// Allow SSO plugins to override/append options (for use in passport prototype authorizationParams)
({ opts } = await plugins.hooks.fire('filter:auth.options', { req, res, opts }));
passport.authenticate(strategy.name, opts)(req, res, next);
}); });
} }