Merge pull request #210 from Infomaniak/appUpdate-exceptions

Catch PlayServices AppUpdate exceptions to better understand what's happening
Infomaniak · Jul 23, 2024 · b17b710 · b17b710
2 parents 388ee4f + d5ee196
commit b17b710
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/Stores/src/standard/java/com/infomaniak/lib/stores/AppUpdateScheduler.kt b/Stores/src/standard/java/com/infomaniak/lib/stores/AppUpdateScheduler.kt
@@ -24,6 +24,7 @@ import com.google.common.util.concurrent.ListenableFuture
 import com.infomaniak.lib.core.utils.SentryLog
 import com.infomaniak.lib.stores.updatemanagers.WorkerUpdateManager
 import io.sentry.Sentry
+import io.sentry.SentryLevel
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
@@ -78,7 +79,10 @@ class AppUpdateScheduler(
                 updateManager.installDownloadedUpdate(
                     onInstallSuccess = { completer.setResult(Result.success()) },
                     onInstallFailure = { exception ->
-                        Sentry.captureException(exception)
+                        Sentry.withScope {scope ->
+                            scope.setTag("message", exception.message ?: "Unknown error")
+                            Sentry.captureMessage("AppUpdate throwed an exception", SentryLevel.INFO)
+                        }
                         completer.setResult(Result.failure())
                     },
                 )