Merge pull request #714 from SISheogorath/fix/uncaughtException

Fix uncaught exception for non-existent user
2018-01-31 20:48:59 +01:00 · 2018-01-31 20:48:59 +01:00 · 2024262200
parent 1a4800e21a eddf8a3a33
commit 2024262200
1 changed files with 5 additions and 0 deletions
--- a/lib/web/auth/index.js
+++ b/lib/web/auth/index.js
@ -21,6 +21,11 @@ passport.deserializeUser(function (id, done) {
      id: id
    }
  }).then(function (user) {
+    // Don't die on non-existent user
+    if (user == null) {
+      return done(null, false, { message: 'Invalid UserID' })
+    }
+
    logger.info('deserializeUser: ' + user.id)
    return done(null, user)
  }).catch(function (err) {