Skip to content
This repository was archived by the owner on Jan 26, 2025. It is now read-only.

Commit e5edb60

Browse files
sergeyteng-prod-CI-bot-okta
authored andcommitted
fix[oidc-middleware]: Fix success handling
OKTA-388994 <<<Jenkins Check-In of Tested SHA: df00ee8 for eng_productivity_ci_bot_okta@okta.com>>> Artifact: okta-oidc-js Files changed count: 3 PR Link: "#990"
1 parent d3c8a1d commit e5edb60

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

packages/oidc-middleware/src/connectUtil.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -113,11 +113,12 @@ connectUtil.createLoginCallbackHandler = context => {
113113
throw new OIDCMiddlewareError('middlewareError', 'Your custom callback handler must request "next"');
114114
}
115115
};
116-
passport.authenticate("oidc", (err, ok, challenge, status) => {
117-
if (!err && !ok) {
118-
err = challenge;
116+
passport.authenticate("oidc", (err, user, challenge) => {
117+
if (user) {
118+
req.logIn(user, nextHandler);
119+
} else {
120+
nextHandler(err || challenge);
119121
}
120-
nextHandler(err);
121122
})(req, res, nextHandler);
122123
}
123124
};

0 commit comments

Comments
 (0)