Merge pull request #127 from Iterable/bugfix/MOB-580-fix-getAdvertisingId-npe

vbabenkoru · web-flow · commit 9a5f5358e50c · 2019-08-05T18:34:51.000-07:00
[MOB-580] Fix a possible NullPointerException in getAdvertisingId
diff --git a/iterableapi/src/main/java/com/iterable/iterableapi/IterableApi.java b/iterableapi/src/main/java/com/iterable/iterableapi/IterableApi.java
@@ -1315,7 +1315,7 @@ private String getAdvertisingId() {
             IterableLogger.d(TAG, "ClassNotFoundException: Can't track ADID. " +
                     "Check that play-services-ads is added to the dependencies.", e);
         } catch (Exception e) {
-            IterableLogger.w(TAG, e.getMessage());
+            IterableLogger.w(TAG, "Error while fetching advertising ID", e);
         }
         return advertisingId;
     }

Original file line number	Diff line number	Diff line change
`@@ -1315,7 +1315,7 @@ private String getAdvertisingId() {`
`1315`	`1315`	`IterableLogger.d(TAG, "ClassNotFoundException: Can't track ADID. " +`
`1316`	`1316`	`"Check that play-services-ads is added to the dependencies.", e);`
`1317`	`1317`	`} catch (Exception e) {`
`1318`		`- IterableLogger.w(TAG, e.getMessage());`
	`1318`	`+ IterableLogger.w(TAG, "Error while fetching advertising ID", e);`
`1319`	`1319`	`}`
`1320`	`1320`	`return advertisingId;`
`1321`	`1321`	`}`