Browse Source

fix: check for email claim before skipping userinfo endpoint

D. MacAlpine 2 months ago
parent
commit
34b62e71cc
1 changed files with 1 additions and 1 deletions
  1. 1 1
      backend/open_webui/utils/oauth.py

+ 1 - 1
backend/open_webui/utils/oauth.py

@@ -193,7 +193,7 @@ class OAuthManager:
             log.warning(f"OAuth callback error: {e}")
             log.warning(f"OAuth callback error: {e}")
             raise HTTPException(400, detail=ERROR_MESSAGES.INVALID_CRED)
             raise HTTPException(400, detail=ERROR_MESSAGES.INVALID_CRED)
         user_data: UserInfo = token.get("userinfo")
         user_data: UserInfo = token.get("userinfo")
-        if not user_data:
+        if not user_data or "email" not in user_data:
             user_data: UserInfo = await client.userinfo(token=token)
             user_data: UserInfo = await client.userinfo(token=token)
         if not user_data:
         if not user_data:
             log.warning(f"OAuth callback failed, user data is missing: {token}")
             log.warning(f"OAuth callback failed, user data is missing: {token}")