fix: migrate all generic logs to Logger class

Mansi-mParticle · Mansi-mParticle · commit 64867e2f333a · 2025-12-04T16:58:38.000-05:00
diff --git a/android-core/src/androidTest/java/com/mparticle/internal/MParticleJSInterfaceITest.java b/android-core/src/androidTest/java/com/mparticle/internal/MParticleJSInterfaceITest.java
@@ -233,7 +233,7 @@ public void setUserAttribute(String json) {
                     called.value = true;
                     latch.countDown();
                 } catch (JSONException jse) {
-                    jse.printStackTrace();
+                    Logger.error(jse);
                 }
 
             }
@@ -259,7 +259,7 @@ public void removeUserAttribute(String json) {
                     called.value = true;
                     latch.countDown();
                 } catch (JSONException jse) {
-                    jse.printStackTrace();
+                    Logger.error(jse);
                 }
             }
         });
@@ -285,7 +285,7 @@ public void setUserTag(String json) {
                     called.value = true;
                     latch.countDown();
                 } catch (JSONException jse) {
-                    jse.printStackTrace();
+                    Logger.error(jse);
                 }
             }
 
@@ -300,7 +300,7 @@ public void setUserAttribute(String json) {
                     called.value = true;
                     latch.countDown();
                 } catch (JSONException jse) {
-                    jse.printStackTrace();
+                    Logger.error(jse);
                 }
             }
         });
@@ -357,7 +357,7 @@ public void logEvent(String json) {
                     }
                     Logger.error(new JSONObject(json).toString(4));
                 } catch (JSONException e) {
-                    e.printStackTrace();
+                    Logger.error(e);
                 }
             }
         });
@@ -466,7 +466,7 @@ public void logout(String json) {
                     called.value = true;
                     latch.countDown();
                 } catch (JSONException e) {
-                    e.printStackTrace();
+                    Logger.error(e);
                 }
             }
         });
@@ -518,7 +518,7 @@ public void login(String json) {
                     called.value = true;
                     latch.countDown();
                 } catch (JSONException e) {
-                    e.printStackTrace();
+                    Logger.error(e);
                 }
             }
         });
@@ -570,7 +570,7 @@ public void modify(String json) {
                     called.value = true;
                     latch.countDown();
                 } catch (JSONException e) {
-                    e.printStackTrace();
+                    Logger.error(e);
                 }
             }
         });
diff --git a/android-core/src/androidTest/kotlin/com.mparticle/PushRegistrationTest.kt b/android-core/src/androidTest/kotlin/com.mparticle/PushRegistrationTest.kt
@@ -2,6 +2,7 @@ package com.mparticle
 
 import android.content.Context
 import com.mparticle.internal.ConfigManager
+import com.mparticle.internal.Logger
 import com.mparticle.internal.PushRegistrationHelper
 import com.mparticle.internal.PushRegistrationHelper.PushRegistration
 import com.mparticle.networking.Matcher
@@ -209,7 +210,7 @@ class PushRegistrationTest : BaseCleanStartedEachTest() {
                         try {
                             Thread.sleep(10)
                         } catch (e: InterruptedException) {
-                            e.printStackTrace()
+                            Logger.error(e)
                         }
                     }
                     TestingUtils.setFirebasePresent(false, null)
diff --git a/android-core/src/androidTest/kotlin/com.mparticle/SessionMessagesTest.kt b/android-core/src/androidTest/kotlin/com.mparticle/SessionMessagesTest.kt
@@ -5,6 +5,7 @@ import android.os.Looper
 import com.mparticle.internal.AccessUtils
 import com.mparticle.internal.AppStateManager
 import com.mparticle.internal.Constants
+import com.mparticle.internal.Logger
 import com.mparticle.networking.Matcher
 import com.mparticle.networking.MockServer.JSONMatch
 import com.mparticle.testutils.AndroidUtils
@@ -61,7 +62,7 @@ sent sessionId = ${eventObject.getString("id")}""",
                             }
                         }
                     } catch (e: Exception) {
-                        e.printStackTrace()
+                        Logger.error(e)
                         Assert.fail(e.message)
                     }
                     false
diff --git a/android-core/src/androidTest/kotlin/com.mparticle/UploadMessageTest.kt b/android-core/src/androidTest/kotlin/com.mparticle/UploadMessageTest.kt
@@ -5,6 +5,7 @@ import android.os.Looper
 import com.mparticle.internal.AccessUtils
 import com.mparticle.internal.AccessUtils.EmptyMParticleApiClient
 import com.mparticle.internal.Constants
+import com.mparticle.internal.Logger
 import com.mparticle.internal.MPUtility
 import com.mparticle.internal.MParticleApiClientImpl.MPRampException
 import com.mparticle.internal.MParticleApiClientImpl.MPThrottleException
@@ -86,7 +87,7 @@ class UploadMessageTest : BaseCleanStartedEachTest() {
                                 }
                             }
                         } catch (e: Exception) {
-                            e.printStackTrace()
+                            Logger.error(e)
                             Assert.fail(e.toString())
                         }
                     }
diff --git a/android-core/src/androidTest/kotlin/com.mparticle/identity/IdentityApiTest.kt b/android-core/src/androidTest/kotlin/com.mparticle/identity/IdentityApiTest.kt
@@ -6,6 +6,7 @@ import android.os.Looper
 import com.mparticle.MParticle
 import com.mparticle.MParticle.IdentityType
 import com.mparticle.internal.ConfigManager
+import com.mparticle.internal.Logger
 import com.mparticle.networking.Matcher
 import com.mparticle.testutils.AndroidUtils
 import com.mparticle.testutils.BaseCleanStartedEachTest
@@ -64,7 +65,7 @@ class IdentityApiTest : BaseCleanStartedEachTest() {
                     com.mparticle.internal.AccessUtils
                         .awaitMessageHandler()
                 } catch (e: InterruptedException) {
-                    e.printStackTrace()
+                    Logger.error(e)
                 }
                 assertMParticleUserEquals(user, mpid1, identities, null, isLoggedIn)
                 latch.countDown()
diff --git a/android-core/src/androidTest/kotlin/com.mparticle/identity/MParticleIdentityClientImplTest.kt b/android-core/src/androidTest/kotlin/com.mparticle/identity/MParticleIdentityClientImplTest.kt
@@ -4,6 +4,7 @@ import android.os.Handler
 import android.util.MutableBoolean
 import com.mparticle.MParticle
 import com.mparticle.internal.ConfigManager
+import com.mparticle.internal.Logger
 import com.mparticle.internal.MPUtility
 import com.mparticle.networking.MPConnection
 import com.mparticle.networking.MPConnectionTestImpl
@@ -323,7 +324,7 @@ class MParticleIdentityClientImplTest : BaseCleanStartedEachTest() {
                     try {
                         identityClient?.makeUrlRequest(connection, payload, identity)
                     } catch (e: JSONException) {
-                        e.printStackTrace()
+                        Logger.error(e)
                         Assert.fail(e.message)
                     }
                     (connection as MPConnectionTestImpl).responseCode = 202
diff --git a/android-core/src/androidTest/kotlin/com.mparticle/internal/AppStateManagerInstrumentedTest.kt b/android-core/src/androidTest/kotlin/com.mparticle/internal/AppStateManagerInstrumentedTest.kt
@@ -57,7 +57,7 @@ class AppStateManagerInstrumentedTest : BaseCleanStartedEachTest() {
                         }
                         checked[0] = true
                     } catch (e: JSONException) {
-                        e.printStackTrace()
+                        Logger.error(e)
                     }
                 }
             },
@@ -97,7 +97,7 @@ class AppStateManagerInstrumentedTest : BaseCleanStartedEachTest() {
                             latch.countDown()
                         }
                     } catch (e: JSONException) {
-                        e.printStackTrace()
+                        Logger.error(e)
                     }
                 }
             },
diff --git a/android-core/src/androidTest/kotlin/com.mparticle/internal/database/services/ReportingServiceTest.kt b/android-core/src/androidTest/kotlin/com.mparticle/internal/database/services/ReportingServiceTest.kt
@@ -2,6 +2,7 @@ package com.mparticle.internal.database.services
 
 import com.mparticle.internal.Constants
 import com.mparticle.internal.JsonReportingMessage
+import com.mparticle.internal.Logger
 import com.mparticle.testutils.TestingUtils
 import org.json.JSONException
 import org.junit.Assert
@@ -213,7 +214,7 @@ class ReportingServiceTest : BaseMPServiceTest() {
                         .getInt("a random Number")
                         .compareTo(o2.msgObject.getInt("a random Number"))
                 } catch (e: JSONException) {
-                    e.printStackTrace()
+                    Logger.error(e)
                 }
                 -1
             },
@@ -227,7 +228,7 @@ class ReportingServiceTest : BaseMPServiceTest() {
                         .getInt("a random Number")
                         .compareTo(o2.toJson().getInt("a random Number"))
                 } catch (e: JSONException) {
-                    e.printStackTrace()
+                    Logger.error(e)
                 }
                 -1
             },
diff --git a/android-core/src/androidTest/kotlin/com.mparticle/networking/PinningTestHelper.kt b/android-core/src/androidTest/kotlin/com.mparticle/networking/PinningTestHelper.kt
@@ -7,6 +7,7 @@ import com.mparticle.MParticle
 import com.mparticle.identity.MParticleIdentityClient
 import com.mparticle.internal.AccessUtils
 import com.mparticle.internal.Constants
+import com.mparticle.internal.Logger
 import com.mparticle.internal.MParticleApiClient
 import com.mparticle.internal.MParticleApiClientImpl
 import com.mparticle.internal.MParticleApiClientImpl.MPNoConfigException
@@ -43,9 +44,9 @@ class PinningTestHelper internal constructor(
                 ),
             )
         } catch (e: MalformedURLException) {
-            e.printStackTrace()
+            Logger.error(e)
         } catch (e: MPNoConfigException) {
-            e.printStackTrace()
+            Logger.error(e)
         }
         val apiClient: MParticleApiClient = AccessUtils.getApiClient()
         setRequestClient(apiClient, path)
diff --git a/android-core/src/main/java/com/mparticle/MPReceiver.java b/android-core/src/main/java/com/mparticle/MPReceiver.java
@@ -8,6 +8,7 @@
 
 import androidx.annotation.NonNull;
 
+import com.mparticle.internal.Logger;
 import com.mparticle.messaging.MPMessagingAPI;
 import com.mparticle.messaging.ProviderCloudMessage;
 
@@ -62,7 +63,7 @@ public void run() {
                 try {
                     Class.forName("android.os.AsyncTask");
                 } catch (ClassNotFoundException e) {
-                    e.printStackTrace();
+                    Logger.error(e);
                 }
             }
         });
diff --git a/android-core/src/main/java/com/mparticle/MPServiceUtil.java b/android-core/src/main/java/com/mparticle/MPServiceUtil.java
@@ -45,7 +45,7 @@ public void run() {
                 try {
                     Class.forName("android.os.AsyncTask");
                 } catch (ClassNotFoundException e) {
-                    e.printStackTrace();
+                    Logger.error(e);
                 }
             }
         });
diff --git a/android-core/src/main/java/com/mparticle/internal/ConfigManager.java b/android-core/src/main/java/com/mparticle/internal/ConfigManager.java
@@ -370,7 +370,7 @@ public synchronized void configUpToDate() throws JSONException {
                 onConfigLoaded(ConfigType.KIT, kitConfig != combined);
             }
         } catch (JSONException e) {
-            e.printStackTrace();
+            Logger.error(e);
             throw new RuntimeException(e);
         }
     }
diff --git a/android-core/src/main/java/com/mparticle/internal/MPUtility.java b/android-core/src/main/java/com/mparticle/internal/MPUtility.java
@@ -811,13 +811,13 @@ public static String getProp(String key) {
             Method get = SystemProperties.getMethod("get", new Class[]{String.class});
             return get.invoke(SystemProperties, new Object[]{key}).toString();
         } catch (ClassNotFoundException e) {
-            e.printStackTrace();
+            Logger.error(e);
         } catch (NoSuchMethodException e) {
-            e.printStackTrace();
+            Logger.error(e);
         } catch (IllegalAccessException e) {
-            e.printStackTrace();
+            Logger.error(e);
         } catch (InvocationTargetException e) {
-            e.printStackTrace();
+            Logger.error(e);
         }
         return null;
     }
diff --git a/android-core/src/main/java/com/mparticle/internal/database/tables/MParticleDatabaseHelper.java b/android-core/src/main/java/com/mparticle/internal/database/tables/MParticleDatabaseHelper.java
@@ -159,7 +159,7 @@ private void removeGcmTable(SQLiteDatabase db) {
         try {
             db.execSQL("DROP TABLE IF EXISTS gcm_messages");
         } catch (Exception e) {
-            e.printStackTrace();
+            Logger.error(e);
         }
     }
 
diff --git a/android-core/src/main/java/com/mparticle/internal/messages/BaseMPMessageBuilder.java b/android-core/src/main/java/com/mparticle/internal/messages/BaseMPMessageBuilder.java
@@ -7,6 +7,7 @@
 
 import com.mparticle.internal.Constants;
 import com.mparticle.internal.InternalSession;
+import com.mparticle.internal.Logger;
 
 import org.json.JSONArray;
 import org.json.JSONException;
@@ -24,15 +25,15 @@ public BaseMPMessageBuilder(String messageType) {
         try {
             put(Constants.MessageKey.TYPE, messageType);
         } catch (JSONException e) {
-            e.printStackTrace();
+            Logger.error(e);
         }
     }
 
     public BaseMPMessageBuilder timestamp(long timestamp) {
         try {
             put(Constants.MessageKey.TIMESTAMP, timestamp);
         } catch (JSONException e) {
-            e.printStackTrace();
+            Logger.error(e);
         }
         return this;
     }
@@ -50,7 +51,7 @@ public BaseMPMessageBuilder name(String name) {
         try {
             put(Constants.MessageKey.NAME, name);
         } catch (JSONException e) {
-            e.printStackTrace();
+            Logger.error(e);
         }
         return this;
     }
@@ -63,7 +64,7 @@ public BaseMPMessageBuilder attributes(JSONObject attributes) {
                     addEventLengthAttributes(mLength);
                 }
             } catch (JSONException e) {
-                e.printStackTrace();
+                Logger.error(e);
             }
         }
         return this;
@@ -82,7 +83,7 @@ public BaseMPMessageBuilder dataConnection(String dataConnection) {
             try {
                 put(Constants.MessageKey.STATE_INFO_DATA_CONNECTION, dataConnection);
             } catch (JSONException e) {
-                e.printStackTrace();
+                Logger.error(e);
             }
         }
         return this;
@@ -95,7 +96,7 @@ public BaseMPMessageBuilder length(Double length) {
                 put(Constants.MessageKey.EVENT_DURATION, length);
                 addEventLengthAttributes(length);
             } catch (JSONException e) {
-                e.printStackTrace();
+                Logger.error(e);
             }
         }
         return this;
@@ -111,7 +112,7 @@ private void addEventLengthAttributes(Double length) {
                 getJSONObject(Constants.MessageKey.ATTRIBUTES).put("EventLength", Integer.toString(length.intValue()));
             }
         } catch (JSONException e) {
-            e.printStackTrace();
+            Logger.error(e);
         }
     }
 
@@ -127,7 +128,7 @@ public BaseMPMessageBuilder flags(Map<String, List<String>> customFlags) {
                 }
                 put(Constants.MessageKey.EVENT_FLAGS, flagsObject);
             } catch (JSONException e) {
-                e.printStackTrace();
+                Logger.error(e);
             }
         }
         return this;
diff --git a/android-core/src/main/java/com/mparticle/internal/messages/MPEventMessage.java b/android-core/src/main/java/com/mparticle/internal/messages/MPEventMessage.java
@@ -7,6 +7,7 @@
 import com.mparticle.MParticle;
 import com.mparticle.internal.Constants;
 import com.mparticle.internal.InternalSession;
+import com.mparticle.internal.Logger;
 
 import org.json.JSONException;
 
@@ -26,7 +27,7 @@ public BaseMPMessageBuilder customEventType(MParticle.EventType eventType) {
             try {
                 put(Constants.MessageKey.EVENT_TYPE, eventType);
             } catch (JSONException e) {
-                e.printStackTrace();
+                Logger.error(e);
             }
             return this;
         }
diff --git a/android-core/src/main/java/com/mparticle/networking/Certificate.java b/android-core/src/main/java/com/mparticle/networking/Certificate.java
@@ -43,7 +43,7 @@ static Certificate withCertificate(JSONObject jsonObject) {
             String certificate = jsonObject.getString("certificate");
             return new Certificate(alias, certificate);
         } catch (JSONException e) {
-            e.printStackTrace();
+            Logger.error(e);
         }
         return null;
     }
diff --git a/android-core/src/main/java/com/mparticle/networking/NetworkOptions.java b/android-core/src/main/java/com/mparticle/networking/NetworkOptions.java
@@ -68,7 +68,7 @@ public static NetworkOptions withNetworkOptions(@Nullable String jsonString) {
                         .build());
             }
         } catch (Exception e) {
-            e.printStackTrace();
+            Logger.error(e);
         }
         return builder.build();
     }
diff --git a/android-core/src/main/kotlin/com/mparticle/internal/AppStateManager.kt b/android-core/src/main/kotlin/com/mparticle/internal/AppStateManager.kt
diff --git a/android-core/src/main/kotlin/com/mparticle/internal/listeners/InternalListenerManager.kt b/android-core/src/main/kotlin/com/mparticle/internal/listeners/InternalListenerManager.kt
diff --git a/testutils/src/androidTest/java/com/mparticle/LegacyStartupTest.java b/testutils/src/androidTest/java/com/mparticle/LegacyStartupTest.java
diff --git a/testutils/src/main/java/com/mparticle/networking/MockServer.java b/testutils/src/main/java/com/mparticle/networking/MockServer.java
diff --git a/testutils/src/main/java/com/mparticle/testutils/StreamAssert.java b/testutils/src/main/java/com/mparticle/testutils/StreamAssert.java
diff --git a/tooling/custom-lint-rules/src/main/java/com/mparticle/lints/basedetectors/CallScanner.kt b/tooling/custom-lint-rules/src/main/java/com/mparticle/lints/basedetectors/CallScanner.kt

Original file line number	Diff line number	Diff line change
`@@ -233,7 +233,7 @@ public void setUserAttribute(String json) {`
`233`	`233`	`called.value = true;`
`234`	`234`	`latch.countDown();`
`235`	`235`	`} catch (JSONException jse) {`
`236`		`- jse.printStackTrace();`
	`236`	`+ Logger.error(jse);`
`237`	`237`	`}`
`238`	`238`
`239`	`239`	`}`
`@@ -259,7 +259,7 @@ public void removeUserAttribute(String json) {`
`259`	`259`	`called.value = true;`
`260`	`260`	`latch.countDown();`
`261`	`261`	`} catch (JSONException jse) {`
`262`		`- jse.printStackTrace();`
	`262`	`+ Logger.error(jse);`
`263`	`263`	`}`
`264`	`264`	`}`
`265`	`265`	`});`
`@@ -285,7 +285,7 @@ public void setUserTag(String json) {`
`285`	`285`	`called.value = true;`
`286`	`286`	`latch.countDown();`
`287`	`287`	`} catch (JSONException jse) {`
`288`		`- jse.printStackTrace();`
	`288`	`+ Logger.error(jse);`
`289`	`289`	`}`
`290`	`290`	`}`
`291`	`291`
`@@ -300,7 +300,7 @@ public void setUserAttribute(String json) {`
`300`	`300`	`called.value = true;`
`301`	`301`	`latch.countDown();`
`302`	`302`	`} catch (JSONException jse) {`
`303`		`- jse.printStackTrace();`
	`303`	`+ Logger.error(jse);`
`304`	`304`	`}`
`305`	`305`	`}`
`306`	`306`	`});`
`@@ -357,7 +357,7 @@ public void logEvent(String json) {`
`357`	`357`	`}`
`358`	`358`	`Logger.error(new JSONObject(json).toString(4));`
`359`	`359`	`} catch (JSONException e) {`
`360`		`- e.printStackTrace();`
	`360`	`+ Logger.error(e);`
`361`	`361`	`}`
`362`	`362`	`}`
`363`	`363`	`});`
`@@ -466,7 +466,7 @@ public void logout(String json) {`
`466`	`466`	`called.value = true;`
`467`	`467`	`latch.countDown();`
`468`	`468`	`} catch (JSONException e) {`
`469`		`- e.printStackTrace();`
	`469`	`+ Logger.error(e);`
`470`	`470`	`}`
`471`	`471`	`}`
`472`	`472`	`});`
`@@ -518,7 +518,7 @@ public void login(String json) {`
`518`	`518`	`called.value = true;`
`519`	`519`	`latch.countDown();`
`520`	`520`	`} catch (JSONException e) {`
`521`		`- e.printStackTrace();`
	`521`	`+ Logger.error(e);`
`522`	`522`	`}`
`523`	`523`	`}`
`524`	`524`	`});`
`@@ -570,7 +570,7 @@ public void modify(String json) {`
`570`	`570`	`called.value = true;`
`571`	`571`	`latch.countDown();`
`572`	`572`	`} catch (JSONException e) {`
`573`		`- e.printStackTrace();`
	`573`	`+ Logger.error(e);`
`574`	`574`	`}`
`575`	`575`	`}`
`576`	`576`	`});`
Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@ import android.os.Looper`
`5`	`5`	`import com.mparticle.internal.AccessUtils`
`6`	`6`	`import com.mparticle.internal.AppStateManager`
`7`	`7`	`import com.mparticle.internal.Constants`
	`8`	`+import com.mparticle.internal.Logger`
`8`	`9`	`import com.mparticle.networking.Matcher`
`9`	`10`	`import com.mparticle.networking.MockServer.JSONMatch`
`10`	`11`	`import com.mparticle.testutils.AndroidUtils`
`@@ -61,7 +62,7 @@ sent sessionId = ${eventObject.getString("id")}""",`
`61`	`62`	`}`
`62`	`63`	`}`
`63`	`64`	`} catch (e: Exception) {`
`64`		`- e.printStackTrace()`
	`65`	`+ Logger.error(e)`
`65`	`66`	`Assert.fail(e.message)`
`66`	`67`	`}`
`67`	`68`	`false`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ class AppStateManagerInstrumentedTest : BaseCleanStartedEachTest() {`
`57`	`57`	`}`
`58`	`58`	`checked[0] = true`
`59`	`59`	`} catch (e: JSONException) {`
`60`		`- e.printStackTrace()`
	`60`	`+ Logger.error(e)`
`61`	`61`	`}`
`62`	`62`	`}`
`63`	`63`	`},`
`@@ -97,7 +97,7 @@ class AppStateManagerInstrumentedTest : BaseCleanStartedEachTest() {`
`97`	`97`	`latch.countDown()`
`98`	`98`	`}`
`99`	`99`	`} catch (e: JSONException) {`
`100`		`- e.printStackTrace()`
	`100`	`+ Logger.error(e)`
`101`	`101`	`}`
`102`	`102`	`}`
`103`	`103`	`},`
Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@`
`8`	`8`
`9`	`9`	`import androidx.annotation.NonNull;`
`10`	`10`
	`11`	`+import com.mparticle.internal.Logger;`
`11`	`12`	`import com.mparticle.messaging.MPMessagingAPI;`
`12`	`13`	`import com.mparticle.messaging.ProviderCloudMessage;`
`13`	`14`
`@@ -62,7 +63,7 @@ public void run() {`
`62`	`63`	`try {`
`63`	`64`	`Class.forName("android.os.AsyncTask");`
`64`	`65`	`} catch (ClassNotFoundException e) {`
`65`		`- e.printStackTrace();`
	`66`	`+ Logger.error(e);`
`66`	`67`	`}`
`67`	`68`	`}`
`68`	`69`	`});`