Don't log DB connection if config 'logSql' => false

PowerKiKi · PowerKiKi · commit 84d5223a2945 · 2025-06-10T17:26:38.000+02:00
Fixes unil-lettres/dilps-tiresias#189
diff --git a/src/DBAL/Logging/Driver.php b/src/DBAL/Logging/Driver.php
@@ -21,7 +21,9 @@ public function __construct(
 
     public function connect(#[SensitiveParameter] array $params): DriverInterface\Connection
     {
-        _log()->debug('Connecting to DB', $this->maskPassword($params));
+        if ($this->logSql) {
+            _log()->debug('Connecting to DB', $this->maskPassword($params));
+        }
 
         // Don't bother to wrap the connection if we will never log SQL queries to file...
         $connection = parent::connect($params);

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,9 @@ public function __construct(`
`21`	`21`
`22`	`22`	`public function connect(#[SensitiveParameter] array $params): DriverInterface\Connection`
`23`	`23`	`{`
`24`		`- _log()->debug('Connecting to DB', $this->maskPassword($params));`
	`24`	`+ if ($this->logSql) {`
	`25`	`+ _log()->debug('Connecting to DB', $this->maskPassword($params));`
	`26`	`+ }`
`25`	`27`
`26`	`28`	`// Don't bother to wrap the connection if we will never log SQL queries to file...`
`27`	`29`	`$connection = parent::connect($params);`