fix(component-passport): Add falback redirects for external paths

pull/17187/head
Mrugesh Mohapatra 2018-05-20 00:50:55 +05:30
parent f9dd384ea2
commit 764d040553
2 changed files with 43 additions and 1 deletions

View File

@ -92,6 +92,47 @@ export default function setupPassport(app) {
Object.keys(passportProviders).map(function(strategy) {
var config = passportProviders[strategy];
config.session = config.session !== false;
// https://stackoverflow.com/q/37430452
let successRedirect = (req) => {
if (!!req && req.session && req.session.returnTo) {
var returnTo = req.session.returnTo;
delete req.session.returnTo;
return returnTo;
}
return config.successRedirect || '';
};
config.customCallback = !config.redirectWithToken
? null
: function(req, res, next) {
var url = require('url');
passport.authenticate(
strategy,
{session: false},
function(err, user, info) {
if (err) {
return next(err);
}
if (!user) {
return res.redirect(config.failureRedirect);
}
var redirect = url.parse(successRedirect(req), true);
delete redirect.search;
redirect.query = {
/* eslint-disable camelcase */
access_token: info.accessToken.id,
/* eslint-enable camelcase */
userId: user.id.toString()
};
redirect = url.format(redirect);
return res.redirect(redirect);
}
)(req, res, next);
};
configurator.configureProvider(
strategy,
{

View File

@ -1,5 +1,5 @@
const successRedirect = '/settings';
const failureRedirect = '/signin';
const failureRedirect = '/';
const linkSuccessRedirect = '/settings';
const linkFailureRedirect = '/settings';
@ -176,6 +176,7 @@ export default {
callbackURL: '/auth/auth0/callback',
authPath: '/auth/auth0',
callbackPath: '/auth/auth0/callback',
redirectWithToken: false,
successRedirect: successRedirect,
failureRedirect: failureRedirect,
scope: ['openid email'],