wrap doInBackgrond with try/catch

Author: e7mac

AndroidSDKCore/src/main/java/com/leanplum/internal/Request.java

@@ -235,30 +235,34 @@ private void saveRequestForLater(final Map<String, Object> args) {
     LeanplumEventDataManager.executeAsyncTask(new AsyncTask<Void, Void, Void>() {
       @Override
       protected Void doInBackground(Void... params) {
-        synchronized (Request.class) {
-          Context context = Leanplum.getContext();
-          SharedPreferences preferences = context.getSharedPreferences(
-              LEANPLUM, Context.MODE_PRIVATE);
-          SharedPreferences.Editor editor = preferences.edit();
-          long count = LeanplumEventDataManager.getEventsCount();
-          String uuid = preferences.getString(Constants.Defaults.UUID_KEY, null);
-          if (uuid == null || count % MAX_EVENTS_PER_API_CALL == 0) {
-            uuid = UUID.randomUUID().toString();
-            editor.putString(Constants.Defaults.UUID_KEY, uuid);
-            SharedPreferencesUtil.commitChanges(editor);
-          }
-          args.put(UUID_KEY, uuid);
-          LeanplumEventDataManager.insertEvent(JsonConverter.toJson(args));
-
-          dataBaseIndex = count;
-          // Checks if here response and/or error callback for this request. We need to add callbacks to
-          // eventCallbackManager only if here was internet connection, otherwise triggerErrorCallback
-          // will handle error callback for this event.
-          if (response != null || error != null && !Util.isConnected()) {
-            eventCallbackManager.addCallbacks(currentRequest, response, error);
+        try {
+          synchronized (Request.class) {
+            Context context = Leanplum.getContext();
+            SharedPreferences preferences = context.getSharedPreferences(
+                    LEANPLUM, Context.MODE_PRIVATE);
+            SharedPreferences.Editor editor = preferences.edit();
+            long count = LeanplumEventDataManager.getEventsCount();
+            String uuid = preferences.getString(Constants.Defaults.UUID_KEY, null);
+            if (uuid == null || count % MAX_EVENTS_PER_API_CALL == 0) {
+              uuid = UUID.randomUUID().toString();
+              editor.putString(Constants.Defaults.UUID_KEY, uuid);
+              SharedPreferencesUtil.commitChanges(editor);
+            }
+            args.put(UUID_KEY, uuid);
+            LeanplumEventDataManager.insertEvent(JsonConverter.toJson(args));
+
+            dataBaseIndex = count;
+            // Checks if here response and/or error callback for this request. We need to add callbacks to
+            // eventCallbackManager only if here was internet connection, otherwise triggerErrorCallback
+            // will handle error callback for this event.
+            if (response != null || error != null && !Util.isConnected()) {
+              eventCallbackManager.addCallbacks(currentRequest, response, error);
+            }
           }
-          return null;
+        } catch (Throwable t) {
+          Util.handleException(t);
         }
+        return null;
       }
     });
   }
@@ -466,7 +470,11 @@ private void sendNow() {
     Util.executeAsyncTask(true, new AsyncTask<Void, Void, Void>() {
       @Override
       protected Void doInBackground(Void... params) {
-        sendRequests();
+        try {
+          sendRequests();
+        } catch (Throwable t) {
+          Util.handleException(t);
+        }
         return null;
       }
     });