Merge pull request #1837 from hackmdio/bugfix/logout-callback

hackmdio · Dec 26, 2023 · 9790586 · 9790586
2 parents f58c669 + c436af1
commit 9790586
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/lib/auth/index.js b/lib/auth/index.js
@@ -49,10 +49,14 @@ if (config.isEmailEnable) authRouter.use(require('./email'))
 if (config.isOpenIDEnable) authRouter.use(require('./openid'))
 
 // logout
-authRouter.get('/logout', function (req, res) {
+authRouter.get('/logout', function (req, res, next) {
   if (config.debug && req.isAuthenticated()) {
     logger.debug('user logout: ' + req.user.id)
   }
-  req.logout()
-  res.redirect(config.serverURL + '/')
+
+  req.logout((err) => {
+    if (err) { return next(err) }
+
+    res.redirect(config.serverURL + '/')
+  })
 })