diff --git a/backend_api_python/app/routes/auth.py b/backend_api_python/app/routes/auth.py index f25c195..0c4f8b4 100644 --- a/backend_api_python/app/routes/auth.py +++ b/backend_api_python/app/routes/auth.py @@ -379,9 +379,14 @@ def login_with_code(): (user['id'],) ) db.commit() + affected = cur.rowcount cur.close() + if affected == 0: + logger.error(f"Failed to update last_login_at: no rows affected for user_id={user['id']}") + else: + logger.info(f"Updated last_login_at for user_id={user['id']}") except Exception as e: - logger.warning(f"Failed to update last_login_at: {e}") + logger.error(f"Failed to update last_login_at for user_id={user.get('id')}: {e}") # Log login security.log_security_event('login_via_code', user['id'], ip_address, user_agent) diff --git a/backend_api_python/app/services/user_service.py b/backend_api_python/app/services/user_service.py index ecd0004..58d3b0b 100644 --- a/backend_api_python/app/services/user_service.py +++ b/backend_api_python/app/services/user_service.py @@ -170,9 +170,14 @@ class UserService: (user['id'],) ) db.commit() + affected = cur.rowcount cur.close() + if affected == 0: + logger.error(f"Failed to update last_login_at: no rows affected for user_id={user['id']}") + else: + logger.info(f"Updated last_login_at for user_id={user['id']}") except Exception as e: - logger.warning(f"Failed to update last_login_at: {e}") + logger.error(f"Failed to update last_login_at for user_id={user.get('id')}: {e}") # Remove password_hash from return value user.pop('password_hash', None)