[android] #2421 - Changing sessionId to UUID and adding flag to change it after 24 hours

1 files changed, 4 insertions, 16 deletions

diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/telemetry/MapboxEventManager.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/telemetry/MapboxEventManager.java
index d909ddab4c..9e79a98036 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/telemetry/MapboxEventManager.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/telemetry/MapboxEventManager.java
@@ -68,6 +68,7 @@ public class MapboxEventManager {
     private String mapboxVendorId = null;
 
     private String mapboxSessionId = null;
+    private long mapboxSessionIdLastSet = 0;
     private static long hourInMillis = 1000 * 60 * 60;
     private static long flushDelayInMillis = 1000 * 60 * 2;  // 2 Minutes
     private static final int SESSION_ID_ROTATION_HOURS = 24;
@@ -185,26 +186,13 @@ public class MapboxEventManager {
     }
 
     private void rotateSessionId() {
-        if (mapboxSessionId == null) {
-            mapboxSessionId = generateNewSessionId();
-            return;
-        }
-
-        // Rotate if it's been SESSION_ID_ROTATION_HOURS hours
-        int start = mapboxSessionId.indexOf("-") + 1;
-        int end = mapboxSessionId.indexOf("-", start);
-        long time = Long.valueOf(mapboxSessionId.substring(start, end));
-
         long now = System.currentTimeMillis();
-        if (now - time > (SESSION_ID_ROTATION_HOURS * hourInMillis)) {
-            mapboxSessionId = generateNewSessionId();
+        if (now - mapboxSessionIdLastSet > (SESSION_ID_ROTATION_HOURS * hourInMillis)) {
+            mapboxSessionId = UUID.randomUUID().toString();
+            mapboxSessionIdLastSet = System.currentTimeMillis();
         }
     }
 
-    private String generateNewSessionId() {
-        return context.getPackageName() + "-" + System.currentTimeMillis() + "-" + UUID.randomUUID().toString();
-    }
-
     private String getOrientation() {
         switch (context.getResources().getConfiguration().orientation) {
             case Configuration.ORIENTATION_LANDSCAPE: