Updated logger

Author: andrej-jasso

GoodNetworking-Sample/GoodNetworking-Sample.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved

@@ -74,9 +74,8 @@ public actor DefaultSessionProvider: NetworkSessionProviding {
     /// - Returns: `true`, indicating the session is valid.
     public var isSessionValid: Bool {
         logger.log(
-            level: .debug,
             message: "✅ Default session is always valid",
-            privacy: .auto
+            level: .debug
         )
         return true
     }
@@ -86,9 +85,8 @@ public actor DefaultSessionProvider: NetworkSessionProviding {
     /// Since the default session does not support invalidation, this method simply logs a message without performing any action.
     public func invalidateSession() async {
         logger.log(
-            level: .debug,
             message: "❌ Default session cannot be invalidated",
-            privacy: .auto
+            level: .debug
         )
     }
 
@@ -100,9 +98,8 @@ public actor DefaultSessionProvider: NetworkSessionProviding {
     /// - Returns: A new instance of `Alamofire.Session`.
     public func makeSession() async -> Alamofire.Session {
         logger.log(
-            level: .debug,
             message: "❌ Default Session Provider cannot be create a new Session, it's setup in the initializer",
-            privacy: .auto
+            level: .debug
         )
 
         return currentSession
@@ -116,9 +113,8 @@ public actor DefaultSessionProvider: NetworkSessionProviding {
     /// - Returns: The current or newly created `Alamofire.Session`.
     public func resolveSession() async -> Alamofire.Session {
         logger.log(
-            level: .debug,
             message: "❌ Default session provider always resolves current session which is setup in the initializer",
-            privacy: .auto
+            level: .debug
         )
         return currentSession
     }

Package.resolved

@@ -56,9 +56,9 @@ public struct LoggingEventMonitor: EventMonitor, Sendable {
 
         switch response.result {
         case .success:
-            logger?.log(level: .debug, message: logMessage, privacy: .auto)
+            logger?.log(message: logMessage, level: .debug, privacy: .auto)
         case .failure:
-            logger?.log(level: .fault, message: logMessage, privacy: .auto)
+            logger?.log(message: logMessage, level: .fault, privacy: .auto)
         }
     }
 

Sources/GoodNetworking/Providers/DefaultSessionProvider.swift

@@ -114,27 +114,27 @@ extension Resource {
     }
 
     public func create() async throws {
-        logger.log(level: .error, message: "CREATE operation not defined for resource \(String(describing: R.self))", privacy: .auto)
+        logger.log(message: "CREATE operation not defined for resource \(String(describing: R.self))", level: .error)
     }
 
     public func read(forceReload: Bool = false) async throws {
-        logger.log(level: .error, message: "READ operation not defined for resource \(String(describing: R.self))", privacy: .auto)
+        logger.log(message: "READ operation not defined for resource \(String(describing: R.self))", level: .error)
     }
 
     public func updateRemote() async throws {
-        logger.log(level: .error, message: "UPDATE operation not defined for resource \(String(describing: R.self))", privacy: .auto)
+        logger.log(message: "UPDATE operation not defined for resource \(String(describing: R.self))", level: .error)
     }
 
     public func delete() async throws {
-        logger.log(level: .error, message: "DELETE operation not defined for resource \(String(describing: R.self))", privacy: .auto)
+        logger.log(message: "DELETE operation not defined for resource \(String(describing: R.self))", level: .error)
     }
 
     public func firstPage(forceReload: Bool = false) async throws {
-        logger.log(level: .error, message: "LIST operation not defined for resource \(String(describing: R.self))", privacy: .auto)
+        logger.log(message: "LIST operation not defined for resource \(String(describing: R.self))", level: .error)
     }
 
     public func nextPage() async throws {
-        logger.log(level: .error, message: "LIST operation not defined for resource \(String(describing: R.self))", privacy: .auto)
+        logger.log(message: "LIST operation not defined for resource \(String(describing: R.self))", level: .error)
     }
 
 }
@@ -146,7 +146,7 @@ extension Resource where R: Creatable {
 
     public func create() async throws {
         guard let request = try R.request(from: state.value) else {
-            return logger.log(level: .error, message: "Creating nil resource always fails! Use create(request:) with a custom request or supply a resource to create.", privacy: .auto)
+            return logger.log(message: "Creating nil resource always fails! Use create(request:) with a custom request or supply a resource to create.", level: .error)
         }
         try await create(request: request)
     }
@@ -197,15 +197,15 @@ extension Resource where R: Readable {
         let resource = state.value
         guard let request = try R.request(from: resource) else {
             self.state = .idle
-            return logger.log(level: .error, message: "Requesting nil resource always fails! Use read(request:forceReload:) with a custom request or supply a resource to read.", privacy: .auto)
+            return logger.log(message: "Requesting nil resource always fails! Use read(request:forceReload:) with a custom request or supply a resource to read.", level: .error)
         }
 
         try await read(request: request, forceReload: forceReload)
     }
 
     public func read(request: R.ReadRequest, forceReload: Bool = false) async throws {
         guard !state.isAvailable || forceReload else {
-            return logger.log(level: .info, message: "Skipping read - value already exists", privacy: .auto)
+            return logger.log(message: "Skipping read - value already exists", level: .info)
         }
 
         let resource = state.value
@@ -245,7 +245,7 @@ extension Resource where R: Updatable {
 
     public func updateRemote() async throws {
         guard let request = try R.request(from: state.value) else {
-            return logger.log(level: .error, message: "Updating resource to nil always fails! Use DELETE instead.", privacy: .auto)
+            return logger.log(message: "Updating resource to nil always fails! Use DELETE instead.", level: .error)
         }
         try await updateRemote(request: request)
     }
@@ -293,7 +293,7 @@ extension Resource where R: Deletable {
 
     public func delete() async throws {
         guard let request = try R.request(from: state.value) else {
-            return logger.log(level: .error, message: "Deleting nil resource always fails. Use delete(request:) with a custom request or supply a resource to delete.", privacy: .auto)
+            return logger.log(message: "Deleting nil resource always fails. Use delete(request:) with a custom request or supply a resource to delete.", level: .error)
         }
         try await delete(request: request)
     }