Merge pull request #9456 from mitomac/fix-oidc-email-claim-new

fix: check for email claim before skipping userinfo endpoint
This commit is contained in:
Timothy Jaeryang Baek 2025-02-05 20:00:35 -08:00 committed by GitHub
commit 397595de9d
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -193,7 +193,7 @@ class OAuthManager:
log.warning(f"OAuth callback error: {e}")
raise HTTPException(400, detail=ERROR_MESSAGES.INVALID_CRED)
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)
if not user_data:
log.warning(f"OAuth callback failed, user data is missing: {token}")