Use structured logging (metadata) when logging queries (vapor#110)

* Don't log double `[]`s when logging queries.

* Use logging metadata for logging queries

Author: gwynne

Sources/SQLiteKit/SQLiteConnection+SQLKit.swift

@@ -218,7 +218,7 @@ struct SQLiteDatabaseVersion: SQLDatabaseReportedVersion {
         let (sql, rawBinds) = self.serialize(query)
 
         if let queryLogLevel = self.queryLogLevel {
-            self.logger.log(level: queryLogLevel, "\(sql) [\(rawBinds)]")
+            self.logger.log(level: queryLogLevel, "Executing query", metadata: ["sql": .string(sql), "binds": .array(rawBinds.map { .string("\($0)") })])
         }
 
         let binds: [SQLiteData]
@@ -244,7 +244,7 @@ struct SQLiteDatabaseVersion: SQLDatabaseReportedVersion {
         let (sql, rawBinds) = self.serialize(query)
 
         if let queryLogLevel = self.queryLogLevel {
-            self.logger.log(level: queryLogLevel, "\(sql) [\(rawBinds)]")
+            self.logger.log(level: queryLogLevel, "Executing query", metadata: ["sql": .string(sql), "binds": .array(rawBinds.map { .string("\($0)") })])
         }
 
         try await self.database.query(

Sources/SQLiteKit/SQLiteConnectionSource.swift

@@ -49,8 +49,7 @@ public struct SQLiteConnectionSource: ConnectionPoolSource, Sendable {
             on: eventLoop
         ).flatMap { conn in
             if self.configuration.enableForeignKeys {
-                return conn.query("PRAGMA foreign_keys = ON")
-                    .map { _ in conn }
+                return conn.query("PRAGMA foreign_keys = ON").map { _ in conn }
             } else {
                 return eventLoop.makeSucceededFuture(conn)
             }