commit ddf1edb35dcc216cd1377300c097470796f6e3ba
parent 475326dd48c9e0d080925eab1b8914824d13f6e4
Author: rhunk <101876869+rhunk@users.noreply.github.com>
Date:   Sun, 28 Apr 2024 18:02:52 +0200

fix(composer_hooks): log crash

Diffstat:

Mcore/src/main/kotlin/me/rhunk/snapenhance/core/bridge/BridgeClient.kt | 2++


Mcore/src/main/kotlin/me/rhunk/snapenhance/core/features/impl/experiments/ComposerHooks.kt | 2+-


2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/core/src/main/kotlin/me/rhunk/snapenhance/core/bridge/BridgeClient.kt b/core/src/main/kotlin/me/rhunk/snapenhance/core/bridge/BridgeClient.kt
@@ -6,6 +6,7 @@ import android.content.Context
 import android.content.Intent
 import android.content.ServiceConnection
 import android.os.*
+import android.util.Log
 import de.robv.android.xposed.XposedHelpers
 import me.rhunk.snapenhance.bridge.AccountStorage
 import me.rhunk.snapenhance.bridge.BridgeInterface
@@ -108,6 +109,7 @@ class BridgeClient(
         return runCatching {
             block()
         }.getOrElse {
+            Log.e("SnapEnhance", "service call failed", it)
             if (it is DeadObjectException) {
                 context.softRestartApp()
             }
diff --git a/core/src/main/kotlin/me/rhunk/snapenhance/core/features/impl/experiments/ComposerHooks.kt b/core/src/main/kotlin/me/rhunk/snapenhance/core/features/impl/experiments/ComposerHooks.kt
@@ -167,7 +167,7 @@ class ComposerHooks: Feature("ComposerHooks", loadParams = FeatureLoadParams.ACT
             "log" -> {
                 if (argc < 3) return false
                 val logLevel = composerMarshaller.getUntyped(1) as? String ?: return false
-                val message = composerMarshaller.getUntyped(2) as? String ?: return false
+                val message = (composerMarshaller.getUntyped(2) as? String)?.takeIf { it.length < 1024 * 512 } ?: return false
 
                 val tag = "ComposerLogs"