Update AGP & provide a logging interface (#13)

Co-authored-by: SessionHero01 <[email protected]>

Author: SessionHero01

gradle/libs.versions.toml

@@ -1,5 +1,5 @@
 [versions]
-agp = "8.10.0"
+agp = "8.10.1"
 kotlin = "2.1.10"
 
 [libraries]

library/src/main/cpp/logging.cpp

@@ -1,47 +1,40 @@
 #include <jni.h>
-#include <android/log.h>
 #include <string_view>
 #include <functional>
 
 #include "session/logging.hpp"
 #include "session/log_level.h"
 
-#define LOG_TAG "LibSession"
-
-extern "C" JNIEXPORT void JNICALL
-Java_network_loki_messenger_libsession_1util_util_Logger_initLogger(JNIEnv* env, jclass clazz) {
-    session::add_logger([](std::string_view msg, std::string_view category, session::LogLevel level) {
-        android_LogPriority prio = ANDROID_LOG_VERBOSE;
-
-        switch (level.level) {
-            case LOG_LEVEL_TRACE:
-                prio = ANDROID_LOG_VERBOSE;
-                break;
-
-            case LOG_LEVEL_DEBUG:
-                prio = ANDROID_LOG_DEBUG;
-                break;
-
-            case LOG_LEVEL_INFO:
-                prio = ANDROID_LOG_INFO;
-                break;
-
-            case LOG_LEVEL_WARN:
-                prio = ANDROID_LOG_WARN;
-                break;
-
-            case LOG_LEVEL_ERROR:
-            case LOG_LEVEL_CRITICAL:
-                prio = ANDROID_LOG_ERROR;
-                break;
-
-            default:
-                prio = ANDROID_LOG_INFO;
-                break;
+#include "jni_utils.h"
+
+using namespace jni_utils;
+
+extern "C"
+JNIEXPORT void JNICALL
+Java_network_loki_messenger_libsession_1util_util_Logger_00024Companion_addLogger(JNIEnv *env, jclass clazz,
+                                                                   jobject logger) {
+    JavaVM *vm = nullptr;
+    env->GetJavaVM(&vm);
+    if (!vm) {
+        env->ThrowNew(env->FindClass("java/lang/RuntimeException"), "Failed to get JavaVM");
+        return;
+    }
+
+    jmethodID logMethod = env->GetMethodID(env->GetObjectClass(logger), "log",
+                                  "(Ljava/lang/String;Ljava/lang/String;I)V");
+
+    auto loggerRef = env->NewGlobalRef(logger);
+    session::add_logger([vm, logMethod, loggerRef](std::string_view msg, std::string_view category, session::LogLevel level) {
+        JNIEnv *env = nullptr;
+        vm->AttachCurrentThread(&env, nullptr);
+        if (!env) {
+            return; // Failed to attach thread, cannot log
         }
 
-        __android_log_print(prio, LOG_TAG, "%.*s [%.*s]",
-                    static_cast<int>(msg.size()), msg.data(),
-                    static_cast<int>(category.size()), category.data());
-});
-}
+        env->CallVoidMethod(loggerRef, logMethod,
+                            JavaLocalRef(env, env->NewStringUTF(msg.data())).get(),
+                            JavaLocalRef(env, env->NewStringUTF(category.data())).get(),
+                            static_cast<jint>(level.level)
+        );
+    });
+}

library/src/main/java/network/loki/messenger/libsession_util/util/Logger.kt

@@ -1,11 +1,23 @@
 package network.loki.messenger.libsession_util.util
 
-object Logger {
+typealias LogLevel = Int
 
-    init {
-        System.loadLibrary("session_util")
-    }
+interface Logger {
+    fun log(message: String, category: String, level: LogLevel)
+
+    companion object {
+        const val LOG_LEVEL_TRACE: LogLevel = 0
+        const val LOG_LEVEL_DEBUG: LogLevel = 1
+        const val LOG_LEVEL_INFO: LogLevel = 2
+        const val LOG_LEVEL_WARN: LogLevel = 3
+        const val LOG_LEVEL_ERROR: LogLevel = 4
+        const val LOG_LEVEL_CRITICAL: LogLevel = 5
+        const val LOG_LEVEL_OFF: LogLevel = 6
 
-    @JvmStatic
-    external fun initLogger()
+        init {
+            System.loadLibrary("session_util")
+        }
+
+        external fun addLogger(logger: Logger)
+    }
 }