Merge pull request matrix-org#1779 from matrix-org/release/0.26.10/re…

…lease

Release 0.26.10

CHANGES.md

@@ -1,3 +1,16 @@
+## Changes in 0.26.10 (2023-05-16)
+
+🙌 Improvements
+
+- Crypto: Enable Crypto SDK by default ([#1770](https://github.com/matrix-org/matrix-ios-sdk/pull/1770))
+- Crypto: Deprecate MXLegacyCrypto ([#1772](https://github.com/matrix-org/matrix-ios-sdk/pull/1772))
+
+🐛 Bugfixes
+
+- Poll: Refreshing the poll when receiving pollEnd can break the chronological order in the store. ([#1776](https://github.com/matrix-org/matrix-ios-sdk/pull/1776))
+- Fix breadcrumb list not updating when leaving a room. Contributed by @JanNikGra. ([#1777](https://github.com/vector-im/element-ios/issues/1777))
+
+
 ## Changes in 0.26.9 (2023-04-18)
 
 🐛 Bugfixes

MatrixSDK.podspec

@@ -1,7 +1,7 @@
 Pod::Spec.new do |s|
 
   s.name         = "MatrixSDK"
-  s.version      = "0.26.9"
+  s.version      = "0.26.10"
   s.summary      = "The iOS SDK to build apps compatible with Matrix (https://www.matrix.org)"
 
   s.description  = <<-DESC

MatrixSDK.xcodeproj/xcshareddata/xcschemes/MatrixSDK-macOS.xcscheme

@@ -64,9 +64,6 @@
          <TestPlanReference
             reference = "container:MatrixSDKTests/TestPlans/UnitTestsWithSanitizers.xctestplan">
          </TestPlanReference>
-         <TestPlanReference
-            reference = "container:MatrixSDKTests/TestPlans/CryptoTests.xctestplan">
-         </TestPlanReference>
          <TestPlanReference
             reference = "container:MatrixSDKTests/TestPlans/AllWorkingTests.xctestplan">
          </TestPlanReference>

MatrixSDK/Aggregations/MXAggregations.m

@@ -315,8 +315,6 @@ - (void)registerListener
                     [self.beaconAggregations handleBeaconWithEvent:event];
                 }
                 break;
-            case MXEventTypePollEnd:
-                [self.aggregatedPollsUpdater refreshPollAfter:event];
             default:
                 break;
         }

MatrixSDK/Background/MXBackgroundSyncService.swift

@@ -67,7 +67,6 @@ public enum MXBackgroundSyncServiceError: Error {
     /// - Parameter credentials: account credentials
     public init(
         withCredentials credentials: MXCredentials,
-        isCryptoSDKEnabled: Bool = false,
         persistTokenDataHandler: MXRestClientPersistTokenDataHandler? = nil,
         unauthenticatedHandler: MXRestClientUnauthenticatedHandler? = nil
     ) {
@@ -90,16 +89,9 @@ public enum MXBackgroundSyncServiceError: Error {
         self.restClient = restClient
 
         store = MXBackgroundStore(withCredentials: credentials)
-        // We can flush any crypto data if our sync response store is empty
-        let resetBackgroundCryptoStore = syncResponseStoreManager.syncToken() == nil
 
-        if isCryptoSDKEnabled {
-            MXLog.debug("[MXBackgroundSyncService] init: constructing crypto v2")
-            crypto = MXBackgroundCryptoV2(credentials: credentials, restClient: restClient)
-        } else {
-            MXLog.debug("[MXBackgroundSyncService] init: constructing legacy crypto")
-            crypto = MXLegacyBackgroundCrypto(credentials: credentials, resetBackgroundCryptoStore: resetBackgroundCryptoStore)
-        }
+        MXLog.debug("[MXBackgroundSyncService] init: constructing crypto")
+        crypto = MXBackgroundCryptoV2(credentials: credentials, restClient: restClient)
 
         pushRulesManager = MXBackgroundPushRulesManager(withCredentials: credentials)
         MXLog.debug("[MXBackgroundSyncService] init complete")

MatrixSDK/Crypto/CrossSigning/MXCrossSigningV2.swift

@@ -126,7 +126,7 @@ class MXCrossSigningV2: NSObject, MXCrossSigning {
                 // If we are considered verified, there is no need for a verification upgrade
                 // after migrating from legacy crypto
                 if myUserCrossSigningKeys?.trustLevel.isVerified == true {
-                    MXSDKOptions.sharedInstance().cryptoSDKFeature?.needsVerificationUpgrade = false
+                    MXSDKOptions.sharedInstance().cryptoMigrationDelegate?.needsVerificationUpgrade = false
                 }
 
                 log.debug("Cross signing state refreshed, new state: \(state)")

MatrixSDK/Crypto/MXCrypto.m

@@ -156,12 +156,6 @@ @implementation MXLegacyCrypto
     __block id<MXCrypto> crypto;
 
 #ifdef MX_CRYPTO
-    if (MXSDKOptions.sharedInstance.enableCryptoSDK)
-    {
-        MXLogFailure(@"[MXCrypto] createCryptoWithMatrixSession: Crypto V2 should not be created directly, use initializeCryptoWithMatrixSession instead");
-        return nil;
-    }
-
     dispatch_queue_t cryptoQueue = [MXLegacyCrypto dispatchQueueForUser:mxSession.matrixRestClient.credentials.userId];
     dispatch_sync(cryptoQueue, ^{
 
@@ -180,22 +174,6 @@ + (void)initializeCryptoWithMatrixSession:(MXSession *)mxSession
                                  complete:(void (^)(id<MXCrypto> crypto, NSError *error))complete
 {
 #ifdef MX_CRYPTO
-
-    // Each time we construct the crypto module (app launch, login etc) we have a chance to try to enable
-    // the newer SDK crypto module, if it is available for this particular user.
-    [MXSDKOptions.sharedInstance.cryptoSDKFeature enableIfAvailableForUserId:mxSession.myUserId];
-    if (MXSDKOptions.sharedInstance.enableCryptoSDK)
-    {
-        [MXCryptoV2Factory.shared buildCryptoWithSession:mxSession
-                                       migrationProgress:migrationProgress
-                                                 success:^(id<MXCrypto> crypto) {
-                                                    complete(crypto, nil); }
-                                                 failure:^(NSError *error) {
-                                                    complete(nil, error);
-                                                 }];
-        return;
-    }
-
     [self initalizeLegacyCryptoWithMatrixSession:mxSession complete:complete];
 #else
     complete(nil);

MatrixSDK/Crypto/MXCryptoV2Factory.swift

@@ -16,6 +16,16 @@
 
 import Foundation
 
+/// Delegate for migrating account data from legacy crypto to rust-based Crypto SDK
+@objc public protocol MXCryptoV2MigrationDelegate {
+
+    /// Flag indicating whether this account requires a re-verification after migrating to Crypto SDK
+    ///
+    /// This flag is set to true if the legacy account is considered verified but the rust account
+    /// does not consider the migrated data secure enough, as it applies stricter security conditions.
+    var needsVerificationUpgrade: Bool { get set }
+}
+
 @objc public class MXCryptoV2Factory: NSObject {
     enum Error: Swift.Error {
         case cryptoNotAvailable
@@ -28,6 +38,21 @@ import Foundation
         .deprecated3
     }
 
+    @objc public func hasCryptoData(for session: MXSession!) -> Bool {
+        guard let userId = session?.myUserId else {
+            log.error("Missing required dependencies")
+            return false
+        }
+
+        do {
+            let url = try MXCryptoMachineStore.storeURL(for: userId)
+            return FileManager.default.fileExists(atPath: url.path)
+        } catch {
+            log.error("Failed creating url for user", context: error)
+            return false
+        }
+    }
+
     @objc public func buildCrypto(
         session: MXSession!,
         migrationProgress: ((Double) -> Void)?,
@@ -124,7 +149,7 @@ import Foundation
             // unless the rust-based crypto already considers the current session to be verified given
             // the migration data
             log.debug("Needs verification upgrade")
-            MXSDKOptions.sharedInstance().cryptoSDKFeature?.needsVerificationUpgrade = true
+            MXSDKOptions.sharedInstance().cryptoMigrationDelegate?.needsVerificationUpgrade = true
         }
     }
 }

MatrixSDK/Data/EventTimeline/Room/MXRoomEventTimeline.m

@@ -142,8 +142,6 @@ - (void)initialiseState:(NSArray<MXEvent *> *)stateEvents
 
 - (void)destroy
 {
-    [room.mxSession resetReplayAttackCheckInTimeline:_timelineId];
-
     if (httpOperation)
     {
         // Cancel the current server request
@@ -190,8 +188,6 @@ - (BOOL)canPaginate:(MXTimelineDirection)direction
 
 - (void)resetPagination
 {
-    [room.mxSession resetReplayAttackCheckInTimeline:_timelineId];
-
     // Reset the back state to the current room state
     backState = [[MXRoomState alloc] initBackStateWith:_state];
 
@@ -203,8 +199,6 @@ - (MXHTTPOperation *)resetPaginationAroundInitialEventWithLimit:(NSUInteger)limi
 {
     NSParameterAssert(success);
     NSAssert(_initialEventId, @"[MXRoomEventTimeline] resetPaginationAroundInitialEventWithLimit cannot be called on live timeline");
-
-    [room.mxSession resetReplayAttackCheckInTimeline:_timelineId];
 
     // Reset the store
     if (!store.isPermanent)

MatrixSDK/Data/EventTimeline/Thread/MXThreadEventTimeline.swift

@@ -99,8 +99,6 @@ public class MXThreadEventTimeline: NSObject, MXEventTimeline {
     }
 
     public func destroy() {
-        thread.session?.resetReplayAttackCheck(inTimeline: timelineId)
-
         removeAllListeners()
 
         currentHttpOperation?.cancel()
@@ -132,8 +130,6 @@ public class MXThreadEventTimeline: NSObject, MXEventTimeline {
     }
 
     public func resetPagination() {
-        thread.session?.resetReplayAttackCheck(inTimeline: timelineId)
-
         // Reset store pagination
         storeMessagesEnumerator = store.messagesEnumerator(forRoom: thread.roomId)
 
@@ -150,8 +146,6 @@ public class MXThreadEventTimeline: NSObject, MXEventTimeline {
             fatalError("[MXThreadEventTimeline][\(timelineId)] resetPaginationAroundInitialEventWithLimit cannot be called on live timeline")
         }
 
-        thread.session?.resetReplayAttackCheck(inTimeline: timelineId)
-
         // Reset the store
         if !store.isPermanent {
             store.deleteAllData()