improvements

Signed-off-by: Marino Faggiana <marino@marinofaggiana.com>

Author: marinofaggiana

Sources/NextcloudKit/NKMonitor.swift

@@ -14,63 +14,68 @@ final class NKMonitor: EventMonitor, Sendable {
     }
 
     func requestDidResume(_ request: Request) {
-        switch NKLogFileManager.shared.logLevel {
-        case .normal:
-            // General-purpose log: full Request description
-            nkLog(info: "Request started: \(request)")
-        case .verbose:
-            // Full dump: headers + body
-            let headers = request.request?.allHTTPHeaderFields?.description ?? "None"
-            let body = request.request?.httpBody.flatMap { String(data: $0, encoding: .utf8) } ?? "None"
-            nkLog(debug: "Request started: \(request)")
-            nkLog(debug: "Headers: \(headers)")
-            nkLog(debug: "Body: \(body)")
-        default:
-            break
+        DispatchQueue.global(qos: .utility).async {
+            switch NKLogFileManager.shared.logLevel {
+            case .normal:
+                // General-purpose log: full Request description
+                nkLog(info: "Request started: \(request)")
+            case .verbose:
+                // Full dump: headers + body
+                let headers = request.request?.allHTTPHeaderFields?.description ?? "None"
+                let body = request.request?.httpBody.flatMap { String(data: $0, encoding: .utf8) } ?? "None"
+                
+                nkLog(debug: "Request started: \(request)")
+                nkLog(debug: "Headers: \(headers)")
+                nkLog(debug: "Body: \(body)")
+            default:
+                break
+            }
         }
     }
 
     func request<Value>(_ request: DataRequest, didParseResponse response: AFDataResponse<Value>) {
         nkCommonInstance.delegate?.request(request, didParseResponse: response)
 
+        // Check for header and account error code tracking
         if let statusCode = response.response?.statusCode,
            let headerCheckInterceptor = request.request?.allHTTPHeaderFields?[nkCommonInstance.headerCheckInterceptor],
            headerCheckInterceptor.lowercased() == "true",
            let account = request.request?.allHTTPHeaderFields?[nkCommonInstance.headerAccount] {
             nkCommonInstance.appendServerErrorAccount(account, errorCode: statusCode)
         }
-        guard let date = self.nkCommonInstance.convertDate(Date(), format: "yyyy-MM-dd' 'HH:mm:ss") else {
-            return
-        }
-        let responseResultString = String("\(response.result)")
-        let responseDebugDescription = String("\(response.debugDescription)")
-        let responseAllHeaderFields = String("\(String(describing: response.response?.allHeaderFields))")
 
-        switch NKLogFileManager.shared.logLevel {
-        case .normal:
-            if let request = response.request {
-                let requestString = "\(request)"
-                nkLog(info: "Network response request: " + requestString + ", result: " + responseResultString)
-            } else {
-                nkLog(info: "Network response result: " + responseResultString)
-            }
-        case .compact:
-            if let method = request.request?.httpMethod,
-               let url = request.request?.url?.absoluteString,
-               let code = response.response?.statusCode {
-                // Determine response status string
-                let responseStatus = (200..<300).contains(code) ? "RESPONSE: SUCCESS" : "RESPONSE: ERROR"
+        let date = nkCommonInstance.convertDate(Date(), format: "yyyy-MM-dd' 'HH:mm:ss") ?? "unknown"
+        let resultString = String(describing: response.result)
+
+        DispatchQueue.global(qos: .utility).async {
+            switch NKLogFileManager.shared.logLevel {
+            case .normal:
+                if let request = response.request {
+                    nkLog(info: "Network response request: \(request), result: \(resultString)")
+                } else {
+                    nkLog(info: "Network response result: \(resultString)")
+                }
+
+            case .compact:
+                if let method = request.request?.httpMethod,
+                   let url = request.request?.url?.absoluteString,
+                   let code = response.response?.statusCode {
+
+                    let responseStatus = (200..<300).contains(code) ? "RESPONSE: SUCCESS" : "RESPONSE: ERROR"
+                    let errorCode = response.error.map { " (\($0._code))" } ?? ""
+                    nkLog(network: "\(code) \(method) \(url) \(responseStatus)\(errorCode)")
+                }
+
+            case .verbose:
+                let debugDesc = String(describing: response)
+                let headerFields = String(describing: response.response?.allHeaderFields ?? [:])
 
-                // Extract error code if any
-                let errorCode = response.error.map { " (\($0._code))" } ?? ""
+                nkLog(debug: "Network response result: \(date) " + debugDesc)
+                nkLog(debug: "Network response all headers: \(date) " + headerFields)
 
-                nkLog(network: "\(code) \(method) \(url) \(responseStatus)\(errorCode)")
+            default:
+                break
             }
-        case .verbose:
-            nkLog(debug: "Network response result: \(date) " + responseDebugDescription)
-            nkLog(debug: "Network response all headers: \(date) " + responseAllHeaderFields)
-        default:
-            break
         }
     }
 }