Просмотр исходного кода

resolved header already sent error

sou 7 лет назад
Родитель
Сommit
78b8fb4dbb
1 измененных файлов с 5 добавлено и 2 удалено
  1. 5 2
      lib/routes/login-passport.js

+ 5 - 2
lib/routes/login-passport.js

@@ -104,6 +104,10 @@ module.exports = function(crowi, app) {
     }
 
     const externalAccount = await getOrCreateUser(req, res, next, userInfo, providerId);
+    if (!externalAccount) {
+      return loginFailure(req, res, next);
+    }
+
     const user = await externalAccount.getPopulatedUser();
 
     // login
@@ -314,10 +318,9 @@ module.exports = function(crowi, app) {
         }
         else {
           req.flash('provider-DuplicatedUsernameException', providerId);
-          return loginFailure(req, res, next);
+          return;
         }
       }
-      // throw err;  // throw again
     }
   }