fix: implemented code review suggestions made by martin

Author: mobilearmor

Plugins/Android/freeRASP.androidlib/src/main/java/com/unity/free/rasp/Controller.java

@@ -1,7 +1,6 @@
 package com.unity.free.rasp;
 
 import android.content.Context;
-import android.util.Log;
 
 import com.unity3d.player.UnityPlayer;
 import com.aheaditec.talsec_security.security.api.SuspiciousAppInfo;
@@ -51,12 +50,12 @@ public void setUnityGameObjectCallback(String gameObjectName) {
 
     @Override
     public void onRootDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onPrivilegedAccess");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onPrivilegedAccess");
     }
 
     @Override
     public void onTamperDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onAppIntegrity");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onAppIntegrity");
     }
 
     @Override
@@ -66,66 +65,66 @@ public void onMalwareDetected(List<SuspiciousAppInfo> list) {
 
     @Override
     public void onDebuggerDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onDebug");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onDebug");
     }
 
     @Override
     public void onEmulatorDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onSimulator");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onSimulator");
     }
 
     @Override
     public void onUntrustedInstallationSourceDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onUnofficialStore");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onUnofficialStore");
     }
 
     @Override
     public void onHookDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onHook");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onRuntimeManipulation");
     }
 
     @Override
     public void onDeviceBindingDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onDeviceBinding");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onDeviceBinding");
     }
 
     @Override
     public void onObfuscationIssuesDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onObfuscationIssues");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onObfuscationIssues");
     }
 
     @Override
     public void onScreenshotDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onScreenshot");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onScreenshot");
     }
 
     @Override
     public void onScreenRecordingDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onScreenRecording");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onScreenRecording");
     }
 
     @Override
     public void onUnlockedDeviceDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onPasscode");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onPasscode");
     }
 
     @Override
     public void onHardwareBackedKeystoreNotAvailableDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onSecureHardwareNotAvailable");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onSecureHardwareNotAvailable");
     }
 
     @Override
     public void onDeveloperModeDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onDevMode");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onDevMode");
     }
 
     @Override
     public void onADBEnabledDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onADBEnabled");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onADBEnabled");
     }
 
     @Override
     public void onSystemVPNDetected() {
-        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResultAndroid", "onSystemVPN");
+        UnityPlayer.UnitySendMessage(this.gameObjectName, "scanResult", "onSystemVPN");
     }       
 }

Plugins/freeRASP/AndroidConfig.cs

@@ -0,0 +1,33 @@
+/**
+ * 
+ * var androidConfig = new AndroidConfig
+ * {
+ *     packageName = "com.example.app",
+ *     signingCertificateHashBase64 = new string[] { "hash1", "hash2" },
+ *     supportedAlternativeStores = new string[] { "store1", "store2" }
+ * };
+ * 
+ **/
+/// <summary>
+/// Struct containing Android-specific parameters for Talsec initialization
+/// </summary>
+[System.Serializable]
+public struct AndroidConfig
+{
+    public string packageName;
+    public string[] signingCertificateHashBase64;
+    public string[] supportedAlternativeStores;
+
+    /// <summary>
+    /// Constructor for AndroidConfig with required packageName
+    /// </summary>
+    /// <param name="packageName">Required Android package name</param>
+    /// <param name="signingCertificateHashBase64">Optional array of signing certificate hashes</param>
+    /// <param name="supportedAlternativeStores">Optional array of supported alternative stores</param>
+    public AndroidConfig(string packageName, string[] signingCertificateHashBase64 = null, string[] supportedAlternativeStores = null)
+    {
+        this.packageName = packageName ?? throw new System.ArgumentNullException(nameof(packageName), "Package name is required");
+        this.signingCertificateHashBase64 = signingCertificateHashBase64;
+        this.supportedAlternativeStores = supportedAlternativeStores;
+    }
+}

Plugins/freeRASP/AndroidConfig.cs.meta

@@ -0,0 +1,33 @@
+/*
+ *
+ * var iosConfig = new IOSConfig
+ * {
+ *     appBundleIds = new string[] { "com.example.app" },
+ *     appTeamId = "TEAM123"
+ * };
+ *
+ * 
+ **/
+/// <summary>
+/// Struct containing iOS-specific parameters for Talsec initialization
+/// </summary>
+[System.Serializable]
+public struct IOSConfig
+{
+    public string[] appBundleIds;
+    public string appTeamId;
+    
+    /// <summary>
+    /// Creates an iOS configuration with bundle IDs and team ID.
+    /// </summary>
+    /// <param name="bundleIds">Required iOS app bundle identifiers</param>
+    /// <param name="teamId">Apple Developer Team ID</param>
+    public IOSConfig(string[] bundleIds, string teamId)
+    {
+        if (bundleIds == null || bundleIds.Length == 0)
+            throw new System.ArgumentException("At least one bundle ID is required");
+        
+        appBundleIds = bundleIds;
+        appTeamId = teamId;
+    }
+}