corrected old env names and fixed logout redirect
This commit is contained in:
@@ -69,7 +69,7 @@ router.get('/login', (req, res, next) => {
|
||||
// router.get('/callback', (req, res, next) => {
|
||||
// passport.authenticate('openidconnect', {
|
||||
// successRedirect: req.session.redirectTo,
|
||||
// failureRedirect: process.env.BASE_URL
|
||||
// failureRedirect: process.env.CLIENT_URL
|
||||
// })
|
||||
// });
|
||||
|
||||
@@ -77,27 +77,27 @@ router.get('/callback', (req, res, next) => {
|
||||
const redirectURI = req.session.redirectTo;
|
||||
passport.authenticate('openidconnect', (err, user) => {
|
||||
if (err) return next(err);
|
||||
if (!user) return res.redirect(process.env.BASE_URL);
|
||||
if (!user) return res.redirect(process.env.CLIENT_URL);
|
||||
|
||||
req.logIn(user, err => {
|
||||
if (err) return next(err);
|
||||
|
||||
// Use redirect saved from session
|
||||
const redirectTo = redirectURI || process.env.BASE_URL;
|
||||
const redirectTo = redirectURI || process.env.CLIENT_URL;
|
||||
delete req.session.redirectTo;
|
||||
return res.redirect(redirectTo);
|
||||
});
|
||||
})(req, res, next);
|
||||
});
|
||||
|
||||
router.post('/logout', function (req, res, next) {
|
||||
router.get('/logout', function (req, res, next) {
|
||||
req.logout(function (err) {
|
||||
if (err) { return next(err); }
|
||||
var params = {
|
||||
client_id: process.env.AUTH_CLIENT_ID,
|
||||
returnTo: process.env.BASE_URL
|
||||
returnTo: process.env.CLIENT_URL
|
||||
};
|
||||
res.redirect(process.env.AUTH_DOMAIN + '/v2/logout?' + querystring.stringify(params));
|
||||
res.redirect(process.env.AUTH_END_SESSION_URI + '?' + querystring.stringify(params));
|
||||
});
|
||||
});
|
||||
|
||||
|
||||
Reference in New Issue
Block a user