cordova-plugin-unvired-logger 0.0.18 → 0.0.19

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "cordova-plugin-unvired-logger",
-    "version": "0.0.18",
+    "version": "0.0.19",
     "description": "A logger plugin for Electron, Android, Browser, and iOS that appends logs to log.txt files organized by user ID.",
     "cordova": {
         "id": "cordova-plugin-unvired-logger",

package/plugin.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<plugin id="cordova-plugin-unvired-logger" version="0.0.18"
+<plugin id="cordova-plugin-unvired-logger" version="0.0.19"
         xmlns="http://apache.org/cordova/ns/plugins/1.0"
         xmlns:android="http://schemas.android.com/apk/res/android">
 

package/src/electron/package.json

@@ -1,6 +1,6 @@
 {
     "name": "cordova-plugin-unvired-logger",
-    "version": "0.0.18",
+    "version": "0.0.19",
     "description": "Electron platform package for cordova-plugin-unvired-logger",
     "main": "Logger.js",
     "scripts": {

package/src/ios/Logger.swift

@@ -6,6 +6,7 @@ import UIKit
     private let LOG_FILE_NAME = "log.txt"
     private let BACKUP_LOG_FILE_NAME = "log_backup.txt"
     private let MAX_LOG_SIZE: Int64 = 10 * 1024 * 1024 // 10MB
+    private let logWriteQueue = DispatchQueue(label: "cordova.plugin.unvired.logger.write")
     
     private var defaultLogLevel: Int = 8
 
@@ -26,8 +27,7 @@ import UIKit
     
     @objc(loggerWithLevel:)
     func loggerWithLevel(_ command: CDVInvokedUrlCommand) {
-        self.commandDelegate.run(inBackground: {
-            guard let args = command.arguments.first as? [String: Any] else {
+        guard let args = command.arguments.first as? [String: Any] else {
                 self.sendErrorResult(command.callbackId, message: "Invalid arguments")
                 return
             }
@@ -58,25 +58,32 @@ import UIKit
             
             do {
                 let logFile = try self.getLogFile(userId: userId)
-                try self.checkAndRotateLogFile(logFile: logFile, userId: userId)
-                
                 let logEntry = self.formatLogEntry(level: level, sourceClass: sourceClass, sourceMethod: sourceMethod, message: message)
                 
                 // Print to console before writing to file
-                print(logEntry.trimmingCharacters(in: .whitespacesAndNewlines))
-                
-                // Perform file write in background
-                self.appendToFile(file: logFile, data: logEntry) { error in
-                    if let error = error {
-                        self.sendErrorResult(command.callbackId, message: "Logging failed: \(error.localizedDescription)")
-                    } else {
-                        self.sendSuccessResult(command.callbackId, message: "Logged to \(logFile.path)")
+                NSLog("%@", logEntry.trimmingCharacters(in: .whitespacesAndNewlines))
+                self.commandDelegate.run(inBackground: {    
+                    // Serialize rotation + write to guarantee ordering
+                    self.logWriteQueue.async {
+                        do {
+                            try self.checkAndRotateLogFile(logFile: logFile, userId: userId)
+                            self.appendToFile(file: logFile, data: logEntry) { error in
+                                if let error = error {
+                                    self.sendErrorResult(command.callbackId, message: "Logging failed: \(error.localizedDescription)")
+                                } else {
+                                    self.sendSuccessResult(command.callbackId, message: "Logged to \(logFile.path)")
+                                }
+                            }
+                        } catch {
+                            self.sendErrorResult(command.callbackId, message: "Logging failed: \(error.localizedDescription)")
+                        }
                     }
-                }
+                })
             } catch {
-                self.sendErrorResult(command.callbackId, message: "Logging failed: \(error.localizedDescription)")
+                self.commandDelegate.run(inBackground: {   
+                    self.sendErrorResult(command.callbackId, message: "Logging failed: \(error.localizedDescription)")
+                })
             }
-        })
     }
     
     @objc(setLogLevel:)
@@ -230,7 +237,9 @@ import UIKit
         let userDir = documentsPath.appendingPathComponent(userId)
         
         if !FileManager.default.fileExists(atPath: userDir.path) {
-            try FileManager.default.createDirectory(at: userDir, withIntermediateDirectories: true, attributes: nil)
+            try FileManager.default.createDirectory(at: userDir, withIntermediateDirectories: true, attributes: [
+                .protectionKey: FileProtectionType.none
+            ])
         }
         
         return userDir
@@ -274,16 +283,38 @@ import UIKit
     private func appendToFile(file: URL, data: String, completion: @escaping (Error?) -> Void = { _ in }) {
         do {
             let fileManager = FileManager.default
+            // Ensure parent directory exists (defensive)
+            let parentDirectory = file.deletingLastPathComponent()
+            if !fileManager.fileExists(atPath: parentDirectory.path) {
+                try fileManager.createDirectory(at: parentDirectory, withIntermediateDirectories: true, attributes: [
+                    .protectionKey: FileProtectionType.none
+                ])
+            }
+            // Create the file if it doesn't exist with safe attributes
             if !fileManager.fileExists(atPath: file.path) {
-                // Create the file if it doesn't exist
-                fileManager.createFile(atPath: file.path, contents: nil, attributes: nil)
+                fileManager.createFile(atPath: file.path, contents: nil, attributes: [
+                    .protectionKey: FileProtectionType.none
+                ])
             }
+            let utf8Data = data.data(using: .utf8) ?? Data()
             let fileHandle = try FileHandle(forWritingTo: file)
-            fileHandle.seekToEndOfFile()
-            fileHandle.write(data.data(using: .utf8)!)
-            fileHandle.closeFile()
+            defer {
+                if #available(iOS 13.0, *) {
+                    try? fileHandle.close()
+                } else {
+                    fileHandle.closeFile()
+                }
+            }
+            if #available(iOS 13.4, *) {
+                try fileHandle.seekToEnd()
+                try fileHandle.write(contentsOf: utf8Data)
+            } else {
+                fileHandle.seekToEndOfFile()
+                fileHandle.write(utf8Data)
+            }
             completion(nil)
         } catch {
+            NSLog("[Logger] Failed to append to file: \(error.localizedDescription)")
             completion(error)
         }
     }
@@ -296,10 +327,14 @@ import UIKit
     }
     
     private func copyFile(from: URL, to: URL) throws {
-        if !FileManager.default.fileExists(atPath: from.path) {
+        let fileManager = FileManager.default
+        if !fileManager.fileExists(atPath: from.path) {
             return
         }
-        try FileManager.default.copyItem(at: from, to: to)
+        if fileManager.fileExists(atPath: to.path) {
+            try fileManager.removeItem(at: to)
+        }
+        try fileManager.copyItem(at: from, to: to)
     }
     
     private func sendSuccessResult(_ callbackId: String, message: String) {