Feature/apm support (#7)

* Feature/apm support flutter (#6)

    * Add setEnabled API for APM

    * Add setColdAppLaunchEnabled API for APM

    * Add setAutoUITraceEnabled API for APM

    * Add support for APM Execution Traces metric

    * Add APIs that start and end UI trace

    * Add specs for APIs that start and end UI trace

    * Fix APM.setEnabled

    * Add setLogLevel API for APM

    * Fix iOS setLogLevel missing variable

    * Fix execution traces APIs

    * Update network logging to support APM for iOS

    * Fix network requests always getting logged as a failure

* Fix APM null safety

* Fix network logger responseHeaders population

* Fix network data model

* Add ignore static member warning

* Add network logging for Android

* Remove unwanted logs

* Fix execution traces specs

* Fix network log payload size for iOS

* Fix InstabugCustomHttpClient request closing

* Call setCurrentPlatform instead of setCrossPlatform

* Adjust APM Network logger for other network libs

* Add debugging log for apmNetworkLog

* Undo comments in InstabugFlutterPlugin.java

* Fix duration in Android bug reporting network logs

* Remove commented code

* Undo SDK versions

Co-authored-by: Ahmed Hisham Rihan <[email protected]>

Author: 2 people

android/build.gradle

@@ -31,6 +31,7 @@ android {
     }
     lintOptions {
         disable 'InvalidPackage'
+        abortOnError false
     }
 }
 dependencies {

android/src/main/java/com/instabug/instabugflutter/ArgsRegistry.java

@@ -7,6 +7,7 @@
 import com.instabug.library.InstabugCustomTextPlaceHolder;
 import com.instabug.library.extendedbugreport.ExtendedBugReport;
 import com.instabug.library.visualusersteps.State;
+import com.instabug.apm.model.LogLevel;
 
 import java.util.HashMap;
 import java.util.Locale;
@@ -60,6 +61,7 @@ final class ArgsRegistry {
         registerInstabugExtendedBugReportModeArgs(ARGS);
         registerInstabugActionTypesArgs(ARGS);
         registerReproStepsModeArgs(ARGS);
+        registerLogLevelArgs(ARGS);
     }
 
     /**
@@ -217,4 +219,13 @@ static void registerReproStepsModeArgs(Map<String, Object> args) {
         args.put("ReproStepsMode.enabledWithNoScreenshots",State.ENABLED_WITH_NO_SCREENSHOTS);
     }
 
+    static void registerLogLevelArgs(Map<String, Object> args) {
+        args.put("logLevelNone", LogLevel.NONE);
+        args.put("logLevelError", LogLevel.ERROR);
+        args.put("logLevelWarning", LogLevel.WARNING);
+        args.put("logLevelInfo", LogLevel.INFO);
+        args.put("logLevelDebug", LogLevel.DEBUG);
+        args.put("logLevelVerbose", LogLevel.VERBOSE);
+    }
+
 }

android/src/main/java/com/instabug/instabugflutter/InstabugFlutterPlugin.java

@@ -7,6 +7,9 @@
 import android.os.Looper;
 import android.util.Log;
 
+import com.instabug.apm.APM;
+import com.instabug.apm.model.ExecutionTrace;
+import com.instabug.apm.networking.APMNetworkLogger;
 import com.instabug.bug.BugReporting;
 import com.instabug.bug.invocation.Option;
 import com.instabug.chat.Chats;
@@ -64,6 +67,7 @@ public class InstabugFlutterPlugin implements MethodCallHandler {
     final public static String INVOCATION_EVENT_SHAKE = "InvocationEvent.shake";
 
     private InstabugCustomTextPlaceHolder placeHolder = new InstabugCustomTextPlaceHolder();
+    HashMap<String, ExecutionTrace> traces = new HashMap<String, ExecutionTrace>();
 
     static MethodChannel channel;
 
@@ -106,14 +110,14 @@ public void onMethodCall(MethodCall call, Result result) {
         }
     }
 
-    private void setCrossPlatform() {
+    private void setCurrentPlatform() {
         try {
-            Method method = getMethod(Class.forName("com.instabug.library.Instabug"), "setCrossPlatform", int.class);
+            Method method = getMethod(Class.forName("com.instabug.library.Instabug"), "setCurrentPlatform", int.class);
             if (method != null) {
-                Log.i("IB-CP-Bridge", "invoking setCrossPlatform with platform: " + Platform.FLUTTER);
+                Log.i("IB-CP-Bridge", "invoking setCurrentPlatform with platform: " + Platform.FLUTTER);
                 method.invoke(null, Platform.FLUTTER);
             } else {
-                Log.e("IB-CP-Bridge", "setCrossPlatform was not found by reflection");
+                Log.e("IB-CP-Bridge", "setCurrentPlatform was not found by reflection");
             }
         } catch (Exception e) {
             e.printStackTrace();
@@ -129,7 +133,7 @@ private void setCrossPlatform() {
      * @param invocationEvents invocationEvents The events that invoke the SDK's UI.
      */
     public void start(Application application, String token, ArrayList<String> invocationEvents) {
-        setCrossPlatform();
+        setCurrentPlatform();
         InstabugInvocationEvent[] invocationEventsArray = new InstabugInvocationEvent[invocationEvents.size()];
         for (int i = 0; i < invocationEvents.size(); i++) {
             String key = invocationEvents.get(i);
@@ -897,7 +901,7 @@ public void networkLog(HashMap<String, Object> jsonObject) throws JSONException
                 (new JSONObject((HashMap<String, String>) jsonObject.get("requestHeaders"))).toString(4));
         networkLog.setResponseHeaders(
                 (new JSONObject((HashMap<String, String>) jsonObject.get("responseHeaders"))).toString(4));
-        networkLog.setTotalDuration(((Number) jsonObject.get("duration")).longValue());
+        networkLog.setTotalDuration(((Number) jsonObject.get("duration")).longValue() / 1000);
         networkLog.insert();
     }
 
@@ -938,6 +942,211 @@ public void run() {
         });
     }
 
+    /**
+     * Enables and disables everything related to APM feature.
+     * 
+     * @param {boolean} isEnabled
+     */
+    public void setAPMEnabled(final boolean isEnabled) {
+        new Handler(Looper.getMainLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                try {
+                    APM.setEnabled(isEnabled);
+                } catch (Exception e) {
+                    e.printStackTrace();
+                }
+            }
+        });
+    }
+
+    /**
+     * Sets the printed logs priority. Filter to one of the following levels.
+     *
+     * @param {String} logLevel.
+     */
+    public void setAPMLogLevel(final String logLevel) {
+        new Handler(Looper.getMainLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                try {
+                    if (ArgsRegistry.getDeserializedValue(logLevel, Integer.class) == null) {
+                        return;
+                    }
+                    APM.setLogLevel((int) ArgsRegistry.getRawValue(logLevel));
+                } catch (Exception e) {
+                    e.printStackTrace();
+                }
+            }
+        });
+    }
+  
+    /**
+     * Enables or disables cold app launch tracking.
+     * @param isEnabled boolean indicating enabled or disabled.
+     */
+    public void setColdAppLaunchEnabled(final boolean isEnabled) {
+        new Handler(Looper.getMainLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                try {
+                    APM.setAppLaunchEnabled(isEnabled);
+                } catch (Exception e) {
+                    e.printStackTrace();
+                }
+            }
+        });
+    }
+    /**
+     * Starts an execution trace
+     * @param name string name of the trace.
+     */
+    public void startExecutionTrace(final String name, final String id) {
+        new Handler(Looper.getMainLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                try {
+                    String result = null;
+                    ExecutionTrace trace = APM.startExecutionTrace(name);
+                    if (trace != null) {
+                        result = id;
+                        traces.put(id, trace);
+                    }
+                    channel.invokeMethod("startExecutionTraceCallBack", result);
+                } catch (Exception e) {
+                    e.printStackTrace();
+                }
+            }
+        });
+    }
+  
+    /**
+     * Sets an execution trace attribute
+     * @param id string id of the trace.
+     * @param key string key of the attribute.
+     * @param value string value of the attribute.
+     */
+    public void setExecutionTraceAttribute(final String id, final String key, final String value) {
+        new Handler(Looper.getMainLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                try {
+                    traces.get(id).setAttribute(key, value);
+                } catch (Exception e) {
+                    e.printStackTrace();
+                }
+            }
+        });
+    }
+  
+    /**
+     * Ends an execution trace
+     * @param id string id of the trace.
+     */
+    public void endExecutionTrace(final String id) {
+        new Handler(Looper.getMainLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                try {
+                    traces.get(id).end();
+                } catch (Exception e) {
+                    e.printStackTrace();
+                }
+            }
+        });
+    }
+  
+    /**
+     * Enables or disables auto UI tracing
+     * @param isEnabled boolean indicating enabled or disabled.
+     */
+    public void setAutoUITraceEnabled(final boolean isEnabled) {
+        new Handler(Looper.getMainLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                try {
+                    APM.setAutoUITraceEnabled(isEnabled);
+                } catch (Exception e) {
+                    e.printStackTrace();
+                }
+            }
+        });
+    }
+
+    /**
+     * Starts a UI trace
+     * @param name string name of the UI trace.
+     */
+    public void startUITrace(final String name) {
+        new Handler(Looper.getMainLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                try {
+                    APM.startUITrace(name);
+                } catch (Exception e) {
+                    e.printStackTrace();
+                }
+            }
+        });
+    }
+
+    /**
+     * Ends the current running UI trace
+     */
+    public void endUITrace() {
+        new Handler(Looper.getMainLooper()).post(new Runnable() {
+            @Override
+            public void run() {
+                try {
+                    APM.endUITrace();
+                } catch (Exception e) {
+                    e.printStackTrace();
+                }
+            }
+        });
+    }
+
+    public void apmNetworkLogByReflection(HashMap<String, Object> jsonObject) throws JSONException {
+        APMNetworkLogger apmNetworkLogger = new APMNetworkLogger();
+        final String requestUrl = (String) jsonObject.get("url");
+        final String requestBody = (String) jsonObject.get("requestBody");
+        final String responseBody = (String) jsonObject.get("responseBody");
+        final String requestMethod = (String) jsonObject.get("method");
+        //--------------------------------------------
+        final String requestContentType = (String) jsonObject.get("contentType");
+        final String responseContentType = (String) jsonObject.get("responseContentType");
+        //--------------------------------------------
+        final String errorDomain = (String) jsonObject.get("errorDomain");
+        final Integer statusCode = (Integer) jsonObject.get("responseCode");
+        final long requestDuration = ((Number) jsonObject.get("duration")).longValue() / 1000;
+        final long requestStartTime = ((Number) jsonObject.get("startTime")).longValue() * 1000;
+        final String requestHeaders = (new JSONObject((HashMap<String, String>) jsonObject.get("requestHeaders"))).toString(4);
+        final String responseHeaders = (new JSONObject((HashMap<String, String>) jsonObject.get("responseHeaders"))).toString(4);
+        final String errorMessage;
+
+        if(errorDomain.equals("")) {
+            errorMessage = null;
+        } else {
+            errorMessage = errorDomain;
+        }
+
+        try {
+            Method method = getMethod(Class.forName("com.instabug.apm.networking.APMNetworkLogger"), "log", long.class, long.class, String.class, String.class, String.class, String.class, String.class, String.class, String.class, int.class, String.class, String.class);
+            if (method != null) {
+                method.invoke(apmNetworkLogger, requestStartTime, requestDuration, requestHeaders, requestBody, requestMethod, requestUrl, requestContentType, responseHeaders, responseBody, statusCode, responseContentType, errorMessage);
+            } else {
+                Log.e("IB-CP-Bridge", "apmNetworkLogByReflection was not found by reflection");
+            }
+        } catch (ClassNotFoundException e) {
+            e.printStackTrace();
+        } catch (IllegalAccessException e) {
+            e.printStackTrace();
+        } catch (InvocationTargetException e) {
+            e.printStackTrace();
+        }
+
+    }
+
     /*
      * 
      * Reports that the screen has been

ios/Classes/InstabugFlutterPlugin.h

@@ -424,4 +424,65 @@
 
 + (void)networkLog:(NSDictionary *)networkData;
 
+/**
+  * Enables and disables everything related to APM feature.
+  * @param isEnabled isEnabled 
+  */
++ (void)setAPMEnabled:(NSNumber *)isEnabled;
+
+/**
+  * Sets the printed logs priority. Filter to one of the following levels:
+  *
+  * - logLevelNone disables all APM SDK console logs.
+  *
+  * - logLevelError prints errors only, we use this level to let you know if something goes wrong.
+  *
+  * - logLevelWarning displays warnings that will not necessarily lead to errors but should be addressed nonetheless.
+  *
+  * - logLevelInfo (default) logs information that we think is useful without being too verbose.
+  *
+  * - logLevelDebug use this in case you are debugging an issue. Not recommended for production use.
+  *
+  * - logLevelVerbose use this only if logLevelDebug was not enough and you need more visibility
+  * on what is going on under the hood.
+  *
+  * Similar to the logLevelDebug level, this is not meant to be used on production environments.
+  *
+  * Each log level will also include logs from all the levels above it. For instance,
+  * logLevelInfo will include logLevelInfo logs as well as logLevelWarning
+  * and logLevelError logs.
+
+  * @param _logLevel the printed logs priority.
+  */
++ (void)setAPMLogLevel:(NSString *)_logLevel;
+
+/**
+  * Enables and disables cold app launch tracking.
+  * @param isEnabled isEnabled 
+  */
++ (void)setColdAppLaunchEnabled:(NSNumber *)isEnabled;
+
++ (void)startExecutionTrace:(NSString *)name id:(NSString *)id;
+
++ (void)setExecutionTraceAttribute:(NSString *)id key:(NSString *)key value:(NSString *)value;
+
++ (void)endExecutionTrace:(NSString *)id;
+
+/**
+  * Enables or disables auto UI tracing.
+  * @param isEnabled boolean indicating enabled or disabled.
+  */
++ (void)setAutoUITraceEnabled:(NSNumber *)isEnabled;
+
+/**
+  * Start UI trace.
+  * @param name string holding the name of the trace.
+  */
++ (void)startUITrace:(NSString *)name;
+
+/**
+  * Ends UI trace.
+  */
++ (void)endUITrace;
+
 @end