Merge dc1a4b7 into 36515a7

NBISweden · Aug 5, 2019 · c8ab3dc · c8ab3dc
2 parents 36515a7 + dc1a4b7
commit c8ab3dc
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/backend/auth.py b/backend/auth.py
@@ -55,10 +55,15 @@ async def get(self):
             user_token = await self.get_user_token(self.get_argument('code'))
             user       = await self.get_user(user_token["access_token"])
 
-            self.set_secure_cookie('access_token', user_token["access_token"])
-            self.set_secure_cookie('user', user["name"])
-            self.set_secure_cookie('email', user["email"])
-            self.set_secure_cookie('identity', user["sub"])
+            try:
+                self.set_secure_cookie('access_token', user_token["access_token"])
+                self.set_secure_cookie('user', user["name"])
+                self.set_secure_cookie('email', user["email"])
+                self.set_secure_cookie('identity', user["sub"])
+            except KeyError as err:
+                logging.error(f'ElixirLoginHandler: data missing ({err}); user: {user}, user_token: {user_token}')
+                self.redirect("/error")
+                return
 
             redirect = self.get_secure_cookie("login_redirect")
             self.clear_cookie("login_redirect")