From 0586c2cf549043330d500e1cefdb2487a2e58d00 Mon Sep 17 00:00:00 2001
From: Stefan Ceriu <stefanc@matrix.org>
Date: Thu, 27 Mar 2025 12:31:38 +0200
Subject: [PATCH] Revert "Attempt to prevent crashes after expiring background
refreshes."
This reverts commit 418daaa90fd9ed6a9fbcdd9d03ee4cde7888c3cd.
---
ElementX/Sources/Application/AppCoordinator.swift | 15 +++------------
1 file changed, 3 insertions(+), 12 deletions(-)
diff --git a/ElementX/Sources/Application/AppCoordinator.swift b/ElementX/Sources/Application/AppCoordinator.swift
index 41e130978..78eb6d9dd 100644
--- a/ElementX/Sources/Application/AppCoordinator.swift
+++ b/ElementX/Sources/Application/AppCoordinator.swift
@@ -893,11 +893,8 @@ class AppCoordinator: AppCoordinatorProtocol, AuthenticationFlowCoordinatorDeleg
// Attempt to stop the background task sync loop cleanly, only if the app not already running
return
}
-
- MainActor.assumeIsolated {
- userSession?.clientProxy.stopSync(completion: completion)
- clientProxyObserver = nil
- }
+ userSession?.clientProxy.stopSync(completion: completion)
+ clientProxyObserver = nil
}
private func startSync() {
@@ -1039,13 +1036,7 @@ class AppCoordinator: AppCoordinatorProtocol, AuthenticationFlowCoordinatorDeleg
// This is important for the app to keep refreshing in the background
scheduleBackgroundAppRefresh()
- /// We have a lot of crashes stemming here which we previously believed are caused by stopSync not being async
- /// on the client proxy side (see the comment on that method). We have now realised that will likely not fix anything but
- /// we also noticed this does not crash on the main thread, even though the whole AppCoordinator is on the Main actor.
- /// As such, we introduced a MainActor conformance on the expirationHandler but we are also assuming main actor
- /// isolated in the `stopSync` method above.
- /// https://sentry.tools.element.io/organizations/element/issues/4477794/
- task.expirationHandler = { @MainActor [weak self] in
+ task.expirationHandler = { [weak self] in
MXLog.info("Background app refresh task is about to expire.")
self?.stopSync(isBackgroundTask: true) {
--
GitLab