Use IO dispatcher for cleanup in bug reporter

element-hq · Jun 25, 2024 · d469cb6 · d469cb6
1 parent ec06ce4
commit d469cb6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/changelog.d/3086.bugfix b/changelog.d/3086.bugfix
@@ -0,0 +1 @@
+Make sure we don't use the main dispatcher while closing the bug report request, as it can lead to crashes in strict mode.
diff --git a/...src/main/kotlin/io/element/android/features/rageshake/impl/reporter/DefaultBugReporter.kt b/...src/main/kotlin/io/element/android/features/rageshake/impl/reporter/DefaultBugReporter.kt
@@ -282,11 +282,13 @@ class DefaultBugReporter @Inject constructor(
                 listener.onUploadFailed(serverError)
             }
         } finally {
-            // delete the generated files when the bug report process has finished
-            for (file in bugReportFiles) {
-                file.safeDelete()
+            withContext(coroutineDispatchers.io) {
+                // delete the generated files when the bug report process has finished
+                for (file in bugReportFiles) {
+                    file.safeDelete()
+                }
+                response?.close()
             }
-            response?.close()
         }
     }
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Make sure we don't use the main dispatcher while closing the bug report request, as it can lead to crashes in strict mode.