diff --git a/server/config/passport.js b/server/config/passport.js index a6cbc5a508..84fcb15f9c 100644 --- a/server/config/passport.js +++ b/server/config/passport.js @@ -123,12 +123,12 @@ passport.use( User.findOne({ github: profile.id }, (findByGithubErr, existingUser) => { if (existingUser) { if (req.user && req.user.email !== existingUser.email) { - done( - new Error('GitHub account is already linked to another account.') - ); + done(null, false, { + msg: 'GitHub account is already linked to another account.' + }); return; } else if (existingUser.banned) { - done(new Error(accountSuspensionMessage)); + done(null, false, { msg: accountSuspensionMessage }); return; } done(null, existingUser); @@ -159,7 +159,7 @@ passport.use( [existingEmailUser] = existingEmailUsers; } if (existingEmailUser.banned) { - done(new Error(accountSuspensionMessage)); + done(null, false, { msg: accountSuspensionMessage }); return; } existingEmailUser.email = existingEmailUser.email || primaryEmail; @@ -218,14 +218,12 @@ passport.use( (findByGoogleErr, existingUser) => { if (existingUser) { if (req.user && req.user.email !== existingUser.email) { - done( - new Error( - 'Google account is already linked to another account.' - ) - ); + done(null, false, { + msg: 'Google account is already linked to another account.' + }); return; } else if (existingUser.banned) { - done(new Error(accountSuspensionMessage)); + done(null, false, { msg: accountSuspensionMessage }); return; } done(null, existingUser); @@ -256,7 +254,7 @@ passport.use( // then, append a random friendly word? if (existingEmailUser) { if (existingEmailUser.banned) { - done(new Error(accountSuspensionMessage)); + done(null, false, { msg: accountSuspensionMessage }); return; } existingEmailUser.email =