diff --git a/.gitignore b/.gitignore index 53dade1..e939057 100644 --- a/.gitignore +++ b/.gitignore @@ -13,6 +13,8 @@ local_postgres_data/ # Carpeta de datos (BD SQLite y similares), pero se mantiene la carpeta data/* !data/.gitkeep +app/data/* +!app/data/.gitkeep *.pyc # Bloquear todos los .env en cualquier carpeta .env diff --git a/app/data/.gitkeep b/app/data/.gitkeep new file mode 100644 index 0000000..e69de29 diff --git a/app/general/utilidades/acciones.py b/app/general/utilidades/acciones.py index 5624a3d..0df937b 100644 --- a/app/general/utilidades/acciones.py +++ b/app/general/utilidades/acciones.py @@ -1,7 +1,9 @@ -from backend_admin.models import Log +import logging from django.utils import timezone from . import utils +logger = logging.getLogger(__name__) + class LogService: """ @@ -11,6 +13,9 @@ class LogService: Uso en vistas: log_id = LogService.gestionar_log(self, request, path='/mi/path/') LogService.gestionar_log(self, request, log_id=log_id, body_request=data, status_code=200) + + Si la BD no está disponible, el log falla silenciosamente para no + interrumpir la petición del usuario. """ def gestionar_log( @@ -23,8 +28,21 @@ class LogService: body_response=None, status_code=None, ): - # Determinar la app que llama a partir del módulo de la vista - modulo = self.__class__.__module__ + try: + return LogService._ejecutar_log( + self, request, log_id, path, user, body_request, body_response, status_code + ) + except Exception as e: + logger.warning('LogService.gestionar_log falló (log omitido): %s', str(e)) + return log_id # Devuelve el log_id existente o None si era creación + + @staticmethod + def _ejecutar_log(caller, request, log_id, path, user, body_request, body_response, status_code): + # Importación aquí para evitar problemas de arranque si la BD no está lista + from backend_admin.models import Log + + # Determinar la app llamante a partir del módulo de la vista + modulo = caller.__class__.__module__ app_nombre = modulo.split('.')[0] tag_header = request.headers.get('tag') @@ -44,20 +62,18 @@ class LogService: # --- CREACIÓN: primer registro del ciclo de vida de la petición --- path_final = path if path else request.path - data_log = { - 'user_id': 0, - 'user': usuario_final, - 'app_id': 0, - 'remote_address': utils.get_client_ip(request), - 'request': '', - 'response': '', - 'status_code': status_code if status_code else '0', - 'path': path_final, - 'method': request.method, - 'createdAt': timezone.now(), - } - - nuevo_log = Log.objects.create(**data_log) + nuevo_log = Log.objects.create( + user_id=0, + user=usuario_final, + app_id=0, + remote_address=utils.get_client_ip(request), + request='', + response='', + status_code=status_code if status_code else '0', + path=path_final, + method=request.method, + createdAt=timezone.now(), + ) return nuevo_log.pk else: