fix: Correction des journaux de connexion et de création de client pour une meilleure traçabilité

Author: remiv1

app_acfc/application.py

@@ -368,7 +368,7 @@ def _apply_successful_login(user: User) -> None:
     elif request.method != 'POST':
         acfc_log.log_to_file(level=WARNING,
                              message=f'{request.method} sur route Login par utilisateur {session.get("user_id", "inconnu")}',
-                             specific_logger=LOG_LOGIN_FILE, zone_log='login', db_log=True)
+                             specific_logger=LOG_LOGIN_FILE, zone_log=LOG_LOGIN_FILE, db_log=True)
         return render_template(ERROR400['page'], title=ERROR400['title'], context=ERROR400['context'], message=WRONG_ROAD)
 
     # === TRAITEMENT POST : Validation des identifiants ===
@@ -377,13 +377,13 @@ def _apply_successful_login(user: User) -> None:
     user = db_session.query(User).filter_by(pseudo=username).first()
     acfc_log.log_to_file(level=INFO,
                          message=f'début de session pour l\'utilisateur: {user is not None}',
-                         specific_logger=LOG_LOGIN_FILE, zone_log='login', db_log=True)
+                         specific_logger=LOG_LOGIN_FILE, zone_log=LOG_LOGIN_FILE, db_log=True)
 
     # Vérification de l'existence de l'utilisateur
     if not user:
         acfc_log.log_to_file(level=WARNING,
                              message=f'Utilisateur non trouvé: {username}',
-                             specific_logger=LOG_LOGIN_FILE, zone_log='login', db_log=True)
+                             specific_logger=LOG_LOGIN_FILE, zone_log=LOG_LOGIN_FILE, db_log=True)
         return render_template(LOGIN['page'], title=LOGIN['title'], context=LOGIN['context'], message=INVALID)
 
     # Vérification du mot de passe avec Argon2... si mot de passe faux
@@ -398,7 +398,7 @@ def _apply_successful_login(user: User) -> None:
         except Exception as e:
             acfc_log.log_to_file(level=ERROR,
                                  message=f'Erreur {e} lors de la validation du mot de passe pour l\'utilisateur: {username}',
-                                 specific_logger=LOG_LOGIN_FILE, zone_log='login', db_log=True)
+                                 specific_logger=LOG_LOGIN_FILE, zone_log=LOG_LOGIN_FILE, db_log=True)
             db_session.rollback()
             return render_template(LOGIN['page'], title=LOGIN['title'], context=LOGIN['context'], message=str(e))
 
@@ -410,7 +410,7 @@ def _apply_successful_login(user: User) -> None:
         db_session.rollback()
         acfc_log.log_to_file(level=ERROR,
                              message=f'Erreur lors de la connexion pour l\'utilisateur: {username}, {e}.',
-                             specific_logger=LOG_LOGIN_FILE, zone_log='login', db_log=True)
+                             specific_logger=LOG_LOGIN_FILE, zone_log=LOG_LOGIN_FILE, db_log=True)
         return render_template(LOGIN['page'], title=LOGIN['title'], context='500', message=str(e))
 
     # Vérification de la nécessité de re-hashage de mot de passe

app_acfc/contextes_bp/clients.py

@@ -363,7 +363,7 @@ def create_client():
             db_session.add(pro)
 
         db_session.commit()
-        acfc_log.log_to_file(logging.INFO, f"Nouveau client créé : ID {nouveau_client.id}", db_log=True, zone_log="clients")
+        acfc_log.log_to_file(logging.INFO, f"Nouveau client créé : ID {nouveau_client.id}", db_log=True, zone_log="clients.log")
 
         return redirect(url_for('clients.get_client', id_client=nouveau_client.id, success_message='Prospect créé avec succès.'))
 
@@ -378,7 +378,7 @@ def create_client():
     except Exception as e:
         if 'db_session' in locals():
             db_session.rollback()
-        acfc_log.log_to_file(logging.ERROR, f"Erreur lors de la création du client : {str(e)}", db_log=True, zone_log="clients")
+        acfc_log.log_to_file(logging.ERROR, f"Erreur lors de la création du client : {str(e)}", db_log=True, zone_log="clients.log")
         return render_template(CLIENT_FORM['page'],
                                title=TITLE_NEW_CLIENT,
                                context=CLIENT_FORM['context'],

clients_api_example.py

@@ -6,16 +6,16 @@
 l'interface de gestion des clients.
 """
 
-from flask import Blueprint, request, jsonify
+from typing import Literal
+from flask import Blueprint, Response, request, jsonify
 from app_acfc.modeles import Client, Part, Pro, Mail, Telephone, Adresse, Commande
 from sqlalchemy import or_, and_
-from sqlalchemy.orm import joinedload
 
 # Blueprint pour les API clients
 clients_api = Blueprint('clients_api', __name__, url_prefix='/api/clients')
 
 @clients_api.route('/search', methods=['GET'])
-def search_clients():
+def search_clients() -> Response | tuple[Response, Literal[500]]:
     """
     Endpoint de recherche de clients
     Paramètres de requête possibles:

logs/fichiers_logs/clients

@@ -36,7 +36,7 @@ def install_test_dependencies():
     print("🔧 Vérification des dépendances de test...")
 
     try:
-        import pytest
+        import pytest   # type: ignore
         print("✅ pytest déjà installé")
     except ImportError:
         print("📦 Installation des dépendances de test...")
@@ -46,7 +46,7 @@ def install_test_dependencies():
         ], check=True)
         print("✅ Dépendances installées")
 
-def run_tests(args):
+def run_tests(args: argparse.Namespace) -> int:
     """Exécute les tests avec les options spécifiées."""
     project_root = get_project_root()
     os.chdir(project_root)
@@ -184,7 +184,7 @@ def main():
 
     # Vérification de pytest
     try:
-        import pytest
+        import pytest   # type: ignore
     except ImportError:
         print("❌ pytest n'est pas installé")
         print("💡 Utilisez --install-deps pour installer les dépendances")