cordova-plugin-unvired-logger 0.0.13 → 0.0.15

package/package.json

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

package/src/android/Logger.java

@@ -14,13 +14,20 @@ import java.text.SimpleDateFormat;
 import java.util.Date;
 import java.util.Locale;
 
+
 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 = 5 * 1024 * 1024; // 5MB
+    private static final long MAX_LOG_SIZE = 10 * 1024 * 1024; // 10MB
+
+    // LogLevel constants as int
+    private static final int LOG_LEVEL_DEBUG = 7;
+    private static final int LOG_LEVEL_ERROR = 8;
+    private static final int LOG_LEVEL_INFO = 9;
+
+    private int defaultLogLevel = LOG_LEVEL_INFO;
 
-    private String defaultLogLevel = "important";
 
     @Override
     public boolean execute(String action, JSONArray args, final CallbackContext callbackContext) throws JSONException {
@@ -36,7 +43,7 @@ public class Logger extends CordovaPlugin {
                     setLogLevel(args, callbackContext);
                     return true;
                 case "getLogLevel":
-                    callbackContext.success(defaultLogLevel);
+                    callbackContext.success(Integer.toString(defaultLogLevel));
                     return true;
                 case "getLogFileContent":
                     getLogFileContent(args, callbackContext);
@@ -67,8 +74,21 @@ public class Logger extends CordovaPlugin {
 
     private void setLogLevel(JSONArray args, CallbackContext callbackContext) {
         try {
-            String level = args.getString(0);
-            defaultLogLevel = level;
+            Object levelObj = args.get(0);
+            int levelInt = LOG_LEVEL_INFO;
+            if (levelObj instanceof Integer) {
+                levelInt = (Integer) levelObj;
+            } else if (levelObj instanceof String) {
+                try {
+                    levelInt = Integer.parseInt((String) levelObj);
+                } catch (NumberFormatException e) {
+                    String levelStr = ((String) levelObj).toLowerCase();
+                    if (levelStr.equals("debug")) levelInt = LOG_LEVEL_DEBUG;
+                    else if (levelStr.equals("error")) levelInt = LOG_LEVEL_ERROR;
+                    else if (levelStr.equals("important")) levelInt = LOG_LEVEL_INFO;
+                }
+            }
+            defaultLogLevel = levelInt;
             callbackContext.success();
         } catch (Exception e) {
             callbackContext.error("Failed to set log level: " + e.getMessage());
@@ -79,13 +99,26 @@ public class Logger extends CordovaPlugin {
         try {
             JSONObject arg = args.getJSONObject(0);
             String userId = arg.optString("userId");
-            String level = arg.optString("level");
+            Object levelObj = arg.opt("level");
+            int level = LOG_LEVEL_INFO;
+            if (levelObj instanceof Integer) {
+                level = (Integer) levelObj;
+            } else if (levelObj instanceof String) {
+                try {
+                    level = Integer.parseInt((String) levelObj);
+                } catch (NumberFormatException e) {
+                    String levelStr = ((String) levelObj).toLowerCase();
+                    if (levelStr.equals("debug")) level = LOG_LEVEL_DEBUG;
+                    else if (levelStr.equals("error")) level = LOG_LEVEL_ERROR;
+                    else if (levelStr.equals("important")) level = LOG_LEVEL_INFO;
+                }
+            }
             String sourceClass = arg.optString("sourceClass", "");
             String sourceMethod = arg.optString("sourceMethod", "");
             String message = arg.optString("message");
 
-            if (userId.isEmpty() || level.isEmpty() || message.isEmpty()) {
-                callbackContext.error("userId, level, and message are required");
+            if (userId.isEmpty() || message.isEmpty()) {
+                callbackContext.error("userId and message are required");
                 return;
             }
 
@@ -107,12 +140,8 @@ public class Logger extends CordovaPlugin {
         }
     }
 
-    private boolean shouldSkipLog(String level) {
-        String normLevel = level.toLowerCase();
-        String normDefault = defaultLogLevel.toLowerCase();
-        if (normDefault.equals("error") && (normLevel.equals("debug") || normLevel.equals("important"))) return true;
-        if (normDefault.equals("important") && normLevel.equals("debug")) return true;
-        return false;
+    private boolean shouldSkipLog(int level) {
+        return level > defaultLogLevel;
     }
 
     private void getLogFileContent(JSONArray args, CallbackContext callbackContext) {
@@ -215,22 +244,23 @@ public class Logger extends CordovaPlugin {
         }
     }
 
-    private String formatLogEntry(String level, String sourceClass, String sourceMethod, String message) {
-        SimpleDateFormat localFormat = new SimpleDateFormat("dd-MM-yyyy HH:mm:ss", Locale.getDefault());
-        SimpleDateFormat utcFormat = new SimpleDateFormat("dd-MM-yyyy HH:mm:ss", Locale.US);
+    private String formatLogEntry(int level, String sourceClass, String sourceMethod, String message) {
+        // Include milliseconds in the format
+        SimpleDateFormat localFormat = new SimpleDateFormat("dd-MM-yyyy HH:mm:ss.SSS", Locale.getDefault());
+        SimpleDateFormat utcFormat = new SimpleDateFormat("dd-MM-yyyy HH:mm:ss.SSS", Locale.US);
         utcFormat.setTimeZone(java.util.TimeZone.getTimeZone("UTC"));
         Date now = new Date();
         String local = localFormat.format(now);
         String utc = utcFormat.format(now);
-        String prefix = level.equalsIgnoreCase("error") ? "⭕️⭕️⭕️ " : "";
+        String prefix = (level == LOG_LEVEL_ERROR) ? "⭕️⭕️⭕️ " : "";
         return prefix + local + " | UTC:" + utc + " | " + getStringFromLevel(level) + " | " + sourceClass + " | " + sourceMethod + " | " + message + "\n";
     }
 
-    private String getStringFromLevel(String level) {
-        switch (level.toLowerCase()) {
-            case "important": return "IMPORTANT";
-            case "error": return "ERROR";
-            case "debug": return "DEBUG";
+    private String getStringFromLevel(int level) {
+        switch (level) {
+            case LOG_LEVEL_INFO: return "IMPORTANT";
+            case LOG_LEVEL_ERROR: return "ERROR";
+            case LOG_LEVEL_DEBUG: return "DEBUG";
             default: return "";
         }
     }

package/src/browser/Logger.js

@@ -1,14 +1,14 @@
 const LOG_FILE_NAME = 'log.txt';
 const BACKUP_LOG_FILE_NAME = 'log_backup.txt';
-const MAX_LOG_SIZE = 5 * 1024 * 1024; // 5MB
+const MAX_LOG_SIZE = 10 * 1024 * 1024; // 10MB
 
 const LogLevel = {
-    Debug: 'debug',
-    Error: 'error',
-    Info: 'important',
+    Debug: 7,
+    Error: 8,
+    Info: 9,
 };
 
-let defaultLogLevel = LogLevel.Info;
+let defaultLogLevel = LogLevel.Info; // Now an int
 
 // Browser storage keys
 const getStorageKey = (userId, fileName) => `unvired_logger_${userId}_${fileName}`;
@@ -29,15 +29,18 @@ module.exports.logInfo = async function (successCallback, errorCallback, args) {
 module.exports.setLogLevel = function(successCallback, errorCallback, level) {
     // If level is an array (from Cordova), extract the first element
     let levelToSet = Array.isArray(level) ? level[0] : level;
-    
-    // Normalize the level to lowercase for consistency
-    defaultLogLevel = (levelToSet || '').toLowerCase();
-    
+    // Convert to int if string
+    if (typeof levelToSet === 'string') {
+        levelToSet = parseInt(levelToSet, 10);
+    }
+    if (Number.isInteger(levelToSet)) {
+        defaultLogLevel = levelToSet;
+    }
     successCallback("");
 }
 
 module.exports.getLogLevel = function(successCallback, errorCallback) {
-    successCallback(defaultLogLevel)
+    successCallback(defaultLogLevel + "")
 }
 
 module.exports.getLogFileURL = function(successCallback, errorCallback, args) {
@@ -114,39 +117,30 @@ async function logWitLevel(successCallback, errorCallback, args)  {
             return errorCallback(new Error("Invalid arguments provided"));
         }
         
-        const { userId, level, sourceClass, sourceMethod, message } = args[0];
+        let { userId, level, sourceClass, sourceMethod, message } = args[0];
         
         if (!userId) return errorCallback(new Error("userId is required"));
-        if (!level) return errorCallback(new Error("level is required"));
+        if (level === undefined || level === null) return errorCallback(new Error("level is required"));
         if (!message) return errorCallback(new Error("message is required"));
 
-        // Normalize log levels for comparison
-        const normalizedDefaultLogLevel = (defaultLogLevel || '').toLowerCase();
-        const normalizedLevel = (level || '').toLowerCase();
-
-        // Log level filtering using normalized values
-        if (
-            (normalizedDefaultLogLevel === LogLevel.Error && (normalizedLevel === LogLevel.Debug || normalizedLevel === LogLevel.Info)) ||
-            (normalizedDefaultLogLevel === LogLevel.Info && normalizedLevel === LogLevel.Debug)
-        ) {
+        // Convert level to int if string
+        if (typeof level === 'string') {
+            level = parseInt(level, 8);
+        }
+        if (level > defaultLogLevel) {
             return successCallback();
         }
 
         await checkAndRotateLogFile(args);
 
         const currentDate = new Date();
-        const formatter = new Intl.DateTimeFormat('en-US', {
-            day: '2-digit', month: '2-digit', year: 'numeric',
-            hour: '2-digit', minute: '2-digit', second: '2-digit'
-        });
-        const localDateString = formatter.format(currentDate);
-
-        const dateUtc = new Date(currentDate.toUTCString());
-        const utcFormatter = new Intl.DateTimeFormat('en-US', {
-            day: '2-digit', month: '2-digit', year: 'numeric',
-            hour: '2-digit', minute: '2-digit', second: '2-digit', timeZone: 'UTC'
-        });
-        const utcDateString = utcFormatter.format(dateUtc);
+        // Format with milliseconds manually
+        function pad(n, z = 2) { return ('00' + n).slice(-z); }
+        const localDateString = `${pad(currentDate.getDate())}-${pad(currentDate.getMonth() + 1)}-${currentDate.getFullYear()} ` +
+            `${pad(currentDate.getHours())}:${pad(currentDate.getMinutes())}:${pad(currentDate.getSeconds())}.${pad(currentDate.getMilliseconds(), 3)}`;
+
+        const utcDateString = `${pad(currentDate.getUTCDate())}-${pad(currentDate.getUTCMonth() + 1)}-${currentDate.getUTCFullYear()} ` +
+            `${pad(currentDate.getUTCHours())}:${pad(currentDate.getUTCMinutes())}:${pad(currentDate.getUTCSeconds())}.${pad(currentDate.getUTCMilliseconds(), 3)}`;
 
         let data = `${localDateString} | UTC:${utcDateString} | ${getStringFromLevel(level)} | ${sourceClass} | ${sourceMethod} | ${message}\n`;
         if (level === LogLevel.Error) {

package/src/electron/Logger.js

@@ -36,13 +36,13 @@ const LOG_FILE_NAME = 'log.txt';
 const BACKUP_LOG_FILE_NAME = 'log_backup.txt';
 
 const LogLevel = {
-    Debug: 'debug',
-    Error: 'error',
-    Info: 'important',
+    Debug: 9,
+    Error: 8,
+    Info: 7,
 };
 
-let defaultLogLevel = LogLevel.Info;
-const MAX_LOG_SIZE = 5 * 1024 * 1024; // 5MB
+let defaultLogLevel = LogLevel.Info; // Now an int
+const MAX_LOG_SIZE = 10 * 1024 * 1024; // 10MB
 
 function logDebug(args) {
     return loggerWithLevel(args);
@@ -59,14 +59,18 @@ function logInfo(args) {
 function setLogLevel(level) {
     // If level is an array (from Cordova), extract the first element
     let levelToSet = Array.isArray(level) ? level[0] : level;
-    
-    // Normalize the level to lowercase for consistency
-    defaultLogLevel = (levelToSet || '').toLowerCase();
+    // Convert to int if string
+    if (typeof levelToSet === 'string') {
+        levelToSet = parseInt(levelToSet, 8);
+    }
+    if (Number.isInteger(levelToSet)) {
+        defaultLogLevel = levelToSet;
+    }
 }
 
 function getLogLevel() {
     return new Promise((resolve) => {
-        resolve(defaultLogLevel);
+        resolve(defaultLogLevel + "");
     });
 }
 
@@ -132,39 +136,31 @@ function copyLogToBackup(args) {
 function loggerWithLevel(args) {
     return withLogLock(async () => {
         if (!args || !args[0]) throw new Error("Invalid arguments provided");
-        const { userId, level, sourceClass, sourceMethod, message } = args[0];
+        let { userId, level, sourceClass, sourceMethod, message } = args[0];
         if (!userId) throw new Error("userId is required");
-        if (!level) throw new Error("level is required");
+        if (level === undefined || level === null) throw new Error("level is required");
         if (!message) throw new Error("message is required");
 
-        // Normalize log levels for comparison
-        const normalizedDefaultLogLevel = (defaultLogLevel || '').toLowerCase();
-        const normalizedLevel = (level || '').toLowerCase();
-
-        // Log level filtering using normalized values
-        if (
-            (normalizedDefaultLogLevel === LogLevel.Error && (normalizedLevel === LogLevel.Debug || normalizedLevel === LogLevel.Info)) ||
-            (normalizedDefaultLogLevel === LogLevel.Info && normalizedLevel === LogLevel.Debug)
-        ) {
-            return;
+        // Convert level to int if string
+        if (typeof level === 'string') {
+            level = parseInt(level, 8);
         }
 
+       if (level < defaultLogLevel) {
+           return;
+       }
+
         await checkAndRotateLogFile(args);
 
         const logPath = getLogFileURL(args);
         const currentDate = new Date();
-        const formatter = new Intl.DateTimeFormat('en-US', {
-            day: '2-digit', month: '2-digit', year: 'numeric',
-            hour: '2-digit', minute: '2-digit', second: '2-digit'
-        });
-        const localDateString = formatter.format(currentDate);
-
-        const dateUtc = new Date(currentDate.toUTCString());
-        const utcFormatter = new Intl.DateTimeFormat('en-US', {
-            day: '2-digit', month: '2-digit', year: 'numeric',
-            hour: '2-digit', minute: '2-digit', second: '2-digit', timeZone: 'UTC'
-        });
-        const utcDateString = utcFormatter.format(dateUtc);
+        // Format with milliseconds manually
+        function pad(n, z = 2) { return ('00' + n).slice(-z); }
+        const localDateString = `${pad(currentDate.getDate())}-${pad(currentDate.getMonth() + 1)}-${currentDate.getFullYear()} ` +
+            `${pad(currentDate.getHours())}:${pad(currentDate.getMinutes())}:${pad(currentDate.getSeconds())}.${pad(currentDate.getMilliseconds(), 3)}`;
+
+        const utcDateString = `${pad(currentDate.getUTCDate())}-${pad(currentDate.getUTCMonth() + 1)}-${currentDate.getUTCFullYear()} ` +
+            `${pad(currentDate.getUTCHours())}:${pad(currentDate.getUTCMinutes())}:${pad(currentDate.getUTCSeconds())}.${pad(currentDate.getUTCMilliseconds(), 3)}`;
 
         let data = `${localDateString} | UTC:${utcDateString} | ${getStringFromLevel(level)} | ${sourceClass} | ${sourceMethod} | ${message}\n`;
         if (level === LogLevel.Error) {

package/src/electron/package.json

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

package/src/ios/Logger.swift

@@ -5,10 +5,11 @@ import UIKit
     
     private let LOG_FILE_NAME = "log.txt"
     private let BACKUP_LOG_FILE_NAME = "log_backup.txt"
-    private let MAX_LOG_SIZE: Int64 = 5 * 1024 * 1024 // 5MB
-    
-    private var defaultLogLevel: String = "important"
+    private let MAX_LOG_SIZE: Int64 = 10 * 1024 * 1024 // 10MB
     
+    private var defaultLogLevel: Int = 8
+    private let fileWriteQueue = DispatchQueue(label: "com.logger.filewrite", qos: .background)
+
     @objc(logDebug:)
     func logDebug(_ command: CDVInvokedUrlCommand) {
         loggerWithLevel(command)
@@ -36,7 +37,7 @@ import UIKit
             return
         }
         
-        guard let level = args["level"] as? String, !level.isEmpty else {
+        guard let level = args["level"] as? Int else {
             sendErrorResult(command.callbackId, message: "level is required")
             return
         }
@@ -60,9 +61,16 @@ import UIKit
             try checkAndRotateLogFile(logFile: logFile, userId: userId)
             
             let logEntry = formatLogEntry(level: level, sourceClass: sourceClass, sourceMethod: sourceMethod, message: message)
-            try appendToFile(file: logFile, data: logEntry)
             
+            // Send success result immediately without waiting for file write
             sendSuccessResult(command.callbackId, message: "Logged to \(logFile.path)")
+            
+            // Perform file write in background with silent error handling
+            appendToFile(file: logFile, data: logEntry) { error in
+                if let error = error {
+                    print("Background logging error: \(error.localizedDescription)")
+                }
+            }
         } catch {
             sendErrorResult(command.callbackId, message: "Logging failed: \(error.localizedDescription)")
         }
@@ -71,7 +79,7 @@ import UIKit
     @objc(setLogLevel:)
     func setLogLevel(_ command: CDVInvokedUrlCommand) {
         guard let args = command.arguments.first as? [String: Any],
-              let level = args["level"] as? String else {
+              let level = args["level"] as? Int else {
             sendErrorResult(command.callbackId, message: "Invalid log level")
             return
         }
@@ -82,7 +90,7 @@ import UIKit
     
     @objc(getLogLevel:)
     func getLogLevel(_ command: CDVInvokedUrlCommand) {
-        sendSuccessResult(command.callbackId, message: defaultLogLevel)
+        sendSuccessResult(command.callbackId, message: "\(defaultLogLevel)")
     }
     
     @objc(getLogFileContent:)
@@ -188,17 +196,8 @@ import UIKit
     
     // MARK: - Helper Methods
     
-    private func shouldSkipLog(level: String) -> Bool {
-        let normLevel = level.lowercased()
-        let normDefault = defaultLogLevel.lowercased()
-        
-        if normDefault == "error" && (normLevel == "debug" || normLevel == "important") {
-            return true
-        }
-        if normDefault == "important" && normLevel == "debug" {
-            return true
-        }
-        return false
+    private func shouldSkipLog(level: Int) -> Bool {
+        return level > defaultLogLevel
     }
     
     private func getLogFile(userId: String) throws -> URL {
@@ -233,40 +232,54 @@ import UIKit
         }
     }
     
-    private func formatLogEntry(level: String, sourceClass: String, sourceMethod: String, message: String) -> String {
+    private func formatLogEntry(level: Int, sourceClass: String, sourceMethod: String, message: String) -> String {
         let dateFormatter = DateFormatter()
-        dateFormatter.dateFormat = "dd-MM-yyyy HH:mm:ss"
+        dateFormatter.dateFormat = "dd-MM-yyyy HH:mm:ss.SSS"
         dateFormatter.locale = Locale.current
         let localDate = dateFormatter.string(from: Date())
         
         let utcFormatter = DateFormatter()
-        utcFormatter.dateFormat = "dd-MM-yyyy HH:mm:ss"
+        utcFormatter.dateFormat = "dd-MM-yyyy HH:mm:ss.SSS"
         utcFormatter.timeZone = TimeZone(abbreviation: "UTC")
         let utcDate = utcFormatter.string(from: Date())
-        
-        let prefix = level.lowercased() == "error" ? "⭕️⭕️⭕️ " : ""
+
+        let prefix = level == 8 ? "⭕️⭕️⭕️ " : ""
         return "\(prefix)\(localDate) | UTC:\(utcDate) | \(getStringFromLevel(level: level)) | \(sourceClass) | \(sourceMethod) | \(message)\n"
     }
-    
-    private func getStringFromLevel(level: String) -> String {
-        switch level.lowercased() {
-        case "important": return "IMPORTANT"
-        case "error": return "ERROR"
-        case "debug": return "DEBUG"
+
+    private func getStringFromLevel(level: Int) -> String {
+        switch level {
+        case 7: return "IMPORTANT"
+        case 8: return "ERROR"
+        case 9: return "DEBUG"
         default: return ""
         }
     }
     
-    private func appendToFile(file: URL, data: String) throws {
-        let fileManager = FileManager.default
-        if !fileManager.fileExists(atPath: file.path) {
-            // Create the file if it doesn't exist
-            fileManager.createFile(atPath: file.path, contents: nil, attributes: nil)
+    private func appendToFile(file: URL, data: String, completion: @escaping (Error?) -> Void = { _ in }) {
+        fileWriteQueue.async {
+            do {
+                let fileManager = FileManager.default
+                if !fileManager.fileExists(atPath: file.path) {
+                    // Create the file if it doesn't exist
+                    fileManager.createFile(atPath: file.path, contents: nil, attributes: nil)
+                }
+                let fileHandle = try FileHandle(forWritingTo: file)
+                fileHandle.seekToEndOfFile()
+                fileHandle.write(data.data(using: .utf8)!)
+                fileHandle.closeFile()
+                
+                // Notify success on main thread
+                DispatchQueue.main.async {
+                    completion(nil)
+                }
+            } catch {
+                // Notify error on main thread
+                DispatchQueue.main.async {
+                    completion(error)
+                }
+            }
         }
-        let fileHandle = try FileHandle(forWritingTo: file)
-        fileHandle.seekToEndOfFile()
-        fileHandle.write(data.data(using: .utf8)!)
-        fileHandle.closeFile()
     }
     
     private func readFile(file: URL) throws -> String {

package/www/logger.js

@@ -2,9 +2,9 @@ var exec = require('cordova/exec');
 
 // LogLevel constants for use in Cordova apps
 exports.LogLevel = {
-    Debug: 'debug',
-    Error: 'error',
-    Info: 'important'
+    Debug: 9,
+    Error: 8,
+    Info: 7
 };
 
 /**
@@ -34,7 +34,17 @@ exports.setLogLevel = function (level, success, error) {
         if (error) error('Log level is required');
         return;
     }
-    exec(success, error, 'Logger', 'setLogLevel', [level]);
+    var logLevel = exports.LogLevel.Error;
+
+    if (level === "DEBUG") {
+        logLevel = exports.LogLevel.Debug;
+    } else if (level === "ERROR") {
+        logLevel = exports.LogLevel.Error;
+    } else if (level === "IMPORTANT") {
+        logLevel = exports.LogLevel.Info;
+    }
+
+    exec(success, error, 'Logger', 'setLogLevel', [logLevel]);
 };
 
 /**