cordova-plugin-unvired-logger 0.0.18 → 0.0.20

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "cordova-plugin-unvired-logger",
-    "version": "0.0.18",
+    "version": "0.0.20",
     "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.20"
         xmlns="http://apache.org/cordova/ns/plugins/1.0"
         xmlns:android="http://schemas.android.com/apk/res/android">
 

package/src/android/Logger.java

@@ -19,7 +19,7 @@ public class Logger extends CordovaPlugin {
 
     private static final String LOG_FILE_NAME = "log.txt";
     private static final String BACKUP_LOG_FILE_NAME = "log_backup.txt";
-    private static final long MAX_LOG_SIZE = 10 * 1024 * 1024; // 10MB
+    private static final long MAX_LOG_SIZE = 50 * 1024 * 1024; // 50MB
 
     // LogLevel constants as int
     private static final int LOG_LEVEL_DEBUG = 7;

package/src/browser/Logger.js

@@ -1,6 +1,6 @@
 const LOG_FILE_NAME = 'log.txt';
 const BACKUP_LOG_FILE_NAME = 'log_backup.txt';
-const MAX_LOG_SIZE = 10 * 1024 * 1024; // 10MB
+const MAX_LOG_SIZE = 50 * 1024 * 1024; // 50MB
 
 const LogLevel = {
     Debug: 7,

package/src/electron/Logger.js

@@ -42,7 +42,7 @@ const LogLevel = {
 };
 
 let defaultLogLevel = LogLevel.Info; // Now an int
-const MAX_LOG_SIZE = 10 * 1024 * 1024; // 10MB
+const MAX_LOG_SIZE = 50 * 1024 * 1024; // 50MB
 
 function logDebug(args) {
     return loggerWithLevel(args);

package/src/electron/package.json

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

package/src/ios/Logger.swift

@@ -5,7 +5,8 @@ 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 MAX_LOG_SIZE: Int64 = 50 * 1024 * 1024 // 50MB
+    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) {