letro/letro-ios
1
0
Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases 2 Wiki Activity

Avoid races between enabling the send queue and receiving back a status update, stop dropping duplicates both here and on reachability status printing.

Browse Source
This commit is contained in:
Stefan Ceriu
2024-06-20 14:59:13 +03:00
committed by Stefan Ceriu
parent 8ece1b00ee
commit 0ec6fba2ba
2 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
ElementX/Sources/Application/AppCoordinator.swift
View File

@@ -665,7 +665,6 @@ class AppCoordinator: AppCoordinatorProtocol, AuthenticationFlowCoordinatorDeleg
let reachabilityNotificationIdentifier = "io.element.elementx.reachability.notification"
ServiceLocator.shared.networkMonitor
.reachabilityPublisher
.removeDuplicates()
.sink { reachability in
MXLog.info("Reachability changed to \(reachability)")

3
ElementX/Sources/Services/Client/ClientProxy.swift
View File

@@ -173,11 +173,10 @@ class ClientProxy: ClientProxyProtocol {
})
sendQueueStatusSubject
.removeDuplicates()
.combineLatest(networkMonitor.reachabilityPublisher)
.debounce(for: 1.0, scheduler: DispatchQueue.main)
.sink { enabled, reachability in
MXLog.info("Send queue status changed to enabled: \(enabled)")
MXLog.info("Send queue status changed to enabled: \(enabled), reachability: \(reachability)")
if enabled == false, reachability == .reachable {
MXLog.info("Enabling all send queues")