Merge pull request #7562 from vector-im/alfogrillo/dm_no_internet_status

Fix offline state issue in user sessions overview

RiotSwiftUI/Modules/UserSessions/UserSessionsOverview/Service/MatrixSDK/UserSessionsDataProvider.swift

@@ -57,8 +57,17 @@ class UserSessionsDataProvider: UserSessionsDataProviderProtocol {
         guard let deviceInfo = deviceInfo else {
             return .permanentlyUnverified
         }
-
-        guard session.crypto?.crossSigning.canCrossSign == true else {
+        
+        // When the app is launched offline the cross signing state is "notBootstrapped"
+        // In this edge case the verification state returned is `.unknown` since we cannot say more even for the current session.
+        guard
+            let crossSigning = session.crypto?.crossSigning,
+            crossSigning.state.rawValue > MXCrossSigningState.notBootstrapped.rawValue
+        else {
+            return .unknown
+        }
+        
+        guard crossSigning.canCrossSign else {
             return deviceInfo.deviceId == session.myDeviceId ? .unverified : .unknown
         }
         

RiotSwiftUI/Modules/UserSessions/UserSessionsOverview/Service/MatrixSDK/UserSessionsOverviewService.swift

@@ -94,7 +94,7 @@ class UserSessionsOverviewService: UserSessionsOverviewServiceProtocol {
         }
         
         overviewDataPublisher = .init(UserSessionsOverviewData(currentSession: currentSessionInfo,
-                                                               unverifiedSessions: currentSessionInfo.verificationState == .verified ? [] : [currentSessionInfo],
+                                                               unverifiedSessions: currentSessionInfo.verificationState.isUnverified ? [currentSessionInfo] : [],
                                                                inactiveSessions: currentSessionInfo.isActive ? [] : [currentSessionInfo],
                                                                otherSessions: [],
                                                                linkDeviceEnabled: false))

RiotTests/UserSessionsDataProviderTests.swift

@@ -209,6 +209,9 @@ private class MockCrossSigning: MXLegacyCrossSigning {
         super.init()
     }
     
+    override var state: MXCrossSigningState {
+        .crossSigningExists
+    }
 }
 
 /// A mock `MXDeviceInfo` that can override the `isVerified` state.

changelog.d/pr-7562.bugfix

@@ -0,0 +1 @@
+Device manager: fix offline state for user's sessions overview.