log id instead of email

Author: avirajsingh7

backend/app/api/routes/credentials.py

@@ -56,7 +56,7 @@ def create_new_credential(*, session: SessionDep, creds_in: CredsCreate):
             project_id=creds_in.project_id,
         )
         if existing_cred:
-            logger.error(
+            logger.warning(
                 f"[create_new_credential] Credentials for provider '{provider}' already exist for organization {creds_in.organization_id} and project {creds_in.project_id}"
             )
             raise HTTPException(
@@ -155,9 +155,6 @@ def delete_provider_credential(
 ):
     provider_enum = validate_provider(provider)
     if not provider_enum:
-        logger.error(
-            f"[delete_provider_credential] Invalid provider: {provider} | org_id: {org_id}, project_id: {project_id}"
-        )
         raise HTTPException(status_code=400, detail="Invalid provider")
     provider_creds = get_provider_credential(
         session=session,

backend/app/api/routes/users.py

@@ -51,7 +51,7 @@ def read_users(session: SessionDep, skip: int = 0, limit: int = 100) -> Any:
 def create_user_endpoint(*, session: SessionDep, user_in: UserCreate) -> Any:
     if get_user_by_email(session=session, email=user_in.email):
         logger.error(
-            f"[create_user_endpoint] Attempt to create user with existing email: {user_in.email}"
+            f"[create_user_endpoint] Attempt to create user with existing email"
         )
         raise HTTPException(
             status_code=400,
@@ -80,7 +80,7 @@ def update_user_me(
         existing_user = get_user_by_email(session=session, email=user_in.email)
         if existing_user and existing_user.id != current_user.id:
             logger.error(
-                f"[update_user_me] Attempt to update user with existing email: {user_in.email}"
+                f"[update_user_me] Attempt to update user with existing email"
             )
             raise HTTPException(
                 status_code=409, detail="User with this email already exists"
@@ -90,7 +90,7 @@ def update_user_me(
     session.add(current_user)
     session.commit()
     session.refresh(current_user)
-    logger.info(f"[update_user_me] User updated with email: {current_user.email}")
+    logger.info(f"[update_user_me] User updated with id: {current_user.id}")
     return current_user
 
 
@@ -110,7 +110,7 @@ def update_password_me(
     current_user.hashed_password = get_password_hash(body.new_password)
     session.add(current_user)
     session.commit()
-    logger.info(f"[update_password_me] Password updated for user: {current_user.email}")
+    logger.info(f"[update_password_me] Password updated for user: {current_user.id}")
     return Message(message="Password updated successfully")
 
 
@@ -128,7 +128,7 @@ def delete_user_me(session: SessionDep, current_user: CurrentUser) -> Any:
         )
     session.delete(current_user)
     session.commit()
-    logger.info(f"[delete_user_me] User deleted: {current_user.email}")
+    logger.info(f"[delete_user_me] User deleted: {current_user.id}")
     return Message(message="User deleted successfully")
 
 
@@ -143,7 +143,7 @@ def register_user(session: SessionDep, user_in: UserRegister) -> Any:
     """
     if get_user_by_email(session=session, email=user_in.email):
         logger.error(
-            f"[register_user] Attempt to create user with existing email: {user_in.email}"
+            f"[register_user] Attempt to create user with existing email"
         )
         raise HTTPException(
             status_code=400,
@@ -195,7 +195,7 @@ def update_user_endpoint(
         existing_user = get_user_by_email(session=session, email=user_in.email)
         if existing_user and existing_user.id != user_id:
             logger.error(
-                f"[update_user_endpoint] Attempt to update user with existing email: {user_in.email}"
+                f"[update_user_endpoint] Attempt to update user with existing email"
             )
             raise HTTPException(
                 status_code=409, detail="User with this email already exists"
@@ -225,5 +225,5 @@ def delete_user(
 
     session.delete(user)
     session.commit()
-    logger.info(f"[delete_user] User deleted: {user.email}")
+    logger.info(f"[delete_user] User deleted: {user.id}")
     return Message(message="User deleted successfully")

backend/app/crud/user.py

@@ -17,7 +17,7 @@ def create_user(*, session: Session, user_create: UserCreate) -> User:
     session.add(db_obj)
     session.commit()
     session.refresh(db_obj)
-    logger.info(f"[create_user] User created with email: {db_obj.email}")
+    logger.info(f"[create_user] User created with id: {db_obj.id}")
     return db_obj
 
 
@@ -33,7 +33,7 @@ def update_user(*, session: Session, db_user: User, user_in: UserUpdate) -> Any:
     session.commit()
     session.refresh(db_user)
     logger.info(
-        f"[update_user] User updated with email: {db_user.email}, updated fields: {user_data.keys()}"
+        f"[update_user] User updated with id: {db_user.id}, updated fields: {user_data.keys()}"
     )
     return db_user