Skip to content

Commit ed18734

Browse files
Use Express next() error handler in AuthCallbackRequest.get
1 parent 16010a4 commit ed18734

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

src/handlers/auth-callback-request.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,18 +21,19 @@ class AuthCallbackRequest {
2121
* router.get('/api/oidc/rp/:issuer_id', AuthCallbackRequest.get)
2222
* ```
2323
*
24-
* @param req
25-
* @param res
24+
* @param req {IncomingRequest}
25+
* @param res {ServerResponse}
26+
* @param next {Function}
2627
*
2728
* @returns {Promise}
2829
*/
29-
static get (req, res) {
30+
static get (req, res, next) {
3031
const request = AuthCallbackRequest.fromParams(req, res)
3132

3233
return AuthCallbackRequest.handle(request)
3334
.catch(error => {
34-
request.debug('Error in AuthCallbackRequest:', error)
35-
res.redirect('/login')
35+
request.debug('Error in AuthCallbackRequest.get:', error)
36+
next(error)
3637
})
3738
}
3839

0 commit comments

Comments
 (0)