fix: logout api should add callback

Signed-off-by: Yukai Huang <yukaihuangtw@gmail.com>
hackmdio · Dec 26, 2023 · c436af1 · c436af1
1 parent 070808b
commit c436af1
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 + '/')
+  })
 })