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

pass req.query.next to sso callback

Browse Source 
user req.query.next instead of req.session.returnTo if it exists
This commit is contained in:
Barış Soner Uşaklı
2023-08-24 21:29:26 -04:00
parent baf38b0d21
commit 2930892aa7
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/routes/authentication.js
View File

@@ -97,6 +97,9 @@ Auth.reloadRoutes = async function (params) {
req.session.ssoState = generateToken(req, true);
opts.state = req.session.ssoState;
}
if (req.query.next) {
req.session.next = req.query.next;
}
// Allow SSO plugins to override/append options (for use in passport prototype authorizationParams)
({ opts } = await plugins.hooks.fire('filter:auth.options', { req, res, opts }));
@@ -116,7 +119,7 @@ Auth.reloadRoutes = async function (params) {
req.session.registration = req.session.registration || {};
// save returnTo for later usage in /register/complete
// passport seems to remove `req.session.returnTo` after it redirects
req.session.registration.returnTo = req.session.returnTo;
req.session.registration.returnTo = req.session.next || req.session.returnTo;
passport.authenticate(strategy.name, (err, user) => {
if (err) {