jcbowyer commited on
Commit
8d0d640
·
verified ·
1 Parent(s): 87ec5a4

Deploy: Consolidated gold tables, fixed nginx docs routing

Browse files
Files changed (1) hide show
  1. api/routes/auth.py +4 -3
api/routes/auth.py CHANGED
@@ -329,11 +329,12 @@ async def oauth_callback(
329
  # Get user info from provider
330
  user_info = await get_user_info(provider, access_token, config)
331
 
332
- if not user_info or not user_info.get('email'):
333
- logger.error(f"Could not retrieve email from {provider}. User info: {user_info}")
 
334
  frontend_url = os.getenv('FRONTEND_URL', '')
335
  redirect_url = oauth_state.redirect_uri or (frontend_url if frontend_url and 'localhost' not in frontend_url else '/')
336
- params = urlencode({'error': f'{provider.title()} login failed: Could not retrieve email'})
337
  return RedirectResponse(url=f"{redirect_url}?{params}")
338
 
339
  except Exception as e:
 
329
  # Get user info from provider
330
  user_info = await get_user_info(provider, access_token, config)
331
 
332
+ # Validate we got user info (email is now always set, even if placeholder for Facebook)
333
+ if not user_info:
334
+ logger.error(f"Could not retrieve user info from {provider}. Check API response logs above.")
335
  frontend_url = os.getenv('FRONTEND_URL', '')
336
  redirect_url = oauth_state.redirect_uri or (frontend_url if frontend_url and 'localhost' not in frontend_url else '/')
337
+ params = urlencode({'error': f'{provider.title()} login failed: Could not retrieve user information'})
338
  return RedirectResponse(url=f"{redirect_url}?{params}")
339
 
340
  except Exception as e: