Skip to content

Commit 46bdaaf

Browse files
db: log SQL and params at DEBUG level in AccessDatabase.execute_query; ensure per-instance logger
1 parent 71bc199 commit 46bdaaf

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

src/common/db/database.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ class AccessDatabase:
3737
def __init__(
3838
self, connection_string: Union[str, Path], pool: Optional[AccessConnectionPool] = None
3939
):
40+
# Logger por instancia para facilitar trazabilidad en tests y producción
41+
self.logger = logging.getLogger(f"{__name__}.AccessDatabase")
4042
# Aceptar Path directo para compatibilidad con tests que pasan ruta en lugar de connection string
4143
if isinstance(connection_string, Path):
4244
# Construir cadena estándar sin password (tests locales)
@@ -127,6 +129,8 @@ def execute_query(
127129
self, query: str, params: Optional[tuple] = None
128130
) -> list[dict[str, Any]]:
129131
if self.pool:
132+
# Log de depuración antes de delegar al pool
133+
self.logger.debug(f"Executing SQL: {query} | Params: {params}")
130134
return self.pool.execute_query(query, params)
131135
if not self._connection:
132136
try:
@@ -140,6 +144,8 @@ def execute_query(
140144
raise
141145
try:
142146
cursor = self._connection.cursor()
147+
# Log de depuración justo antes de ejecutar la consulta real
148+
self.logger.debug(f"Executing SQL: {query} | Params: {params}")
143149
if params:
144150
cursor.execute(query, params)
145151
else:

0 commit comments

Comments
 (0)