Skip to content

Commit 883ae9a

Browse files
committed
remove unused argument
1 parent f983698 commit 883ae9a

File tree

1 file changed

+17
-21
lines changed

1 file changed

+17
-21
lines changed

server/routes/passport.routes.js

Lines changed: 17 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -4,30 +4,26 @@ import passport from 'passport';
44
const router = new Router();
55

66
const authenticateOAuth = (service) => (req, res, next) => {
7-
passport.authenticate(
8-
service,
9-
{ failureRedirect: '/login' },
10-
(err, user, info) => {
11-
if (err) {
12-
// use query string param to show error;
13-
res.redirect(`/account?error=${service}`);
14-
return;
15-
}
7+
passport.authenticate(service, { failureRedirect: '/login' }, (err, user) => {
8+
if (err) {
9+
// use query string param to show error;
10+
res.redirect(`/account?error=${service}`);
11+
return;
12+
}
1613

17-
if (!user) {
18-
res.redirect(`/account?error=${service}NoUser`);
14+
if (!user) {
15+
res.redirect(`/account?error=${service}NoUser`);
16+
return;
17+
}
18+
19+
req.logIn(user, (loginErr) => {
20+
if (loginErr) {
21+
next(loginErr);
1922
return;
2023
}
21-
22-
req.logIn(user, (loginErr) => {
23-
if (loginErr) {
24-
next(loginErr);
25-
return;
26-
}
27-
res.redirect('/');
28-
});
29-
}
30-
)(req, res, next);
24+
res.redirect('/');
25+
});
26+
})(req, res, next);
3127
};
3228

3329
router.get('/auth/github', passport.authenticate('github'));

0 commit comments

Comments
 (0)