Remove dependency on libolm - WIP

tchapgouv · Sep 11, 2024 · 5908cd5 · 5908cd5
1 parent 6a8e978
commit 5908cd5
Show file tree

Hide file tree

Showing 64 changed files with 51 additions and 3,290 deletions.
diff --git a/matrix-sdk-android/build.gradle b/matrix-sdk-android/build.gradle
@@ -205,9 +205,6 @@ dependencies {
     // Work
     implementation libs.androidx.work
 
-    // olm lib is now hosted in MavenCentral
-    implementation 'org.matrix.android:olm-sdk:3.2.12'
-
     // DI
     implementation libs.dagger.dagger
     kapt libs.dagger.daggerCompiler

diff --git a/matrix-sdk-android/src/androidTest/assets/crypto_store_20.realm b/matrix-sdk-android/src/androidTest/assets/crypto_store_20.realm
diff --git a/matrix-sdk-android/src/androidTest/assets/crypto_store_migration_16.realm b/matrix-sdk-android/src/androidTest/assets/crypto_store_migration_16.realm
diff --git a/matrix-sdk-android/src/androidTest/java/org/matrix/android/sdk/common/TestMatrix.kt b/matrix-sdk-android/src/androidTest/java/org/matrix/android/sdk/common/TestMatrix.kt
@@ -36,7 +36,6 @@ import org.matrix.android.sdk.internal.network.ApiInterceptor
 import org.matrix.android.sdk.internal.network.UserAgentHolder
 import org.matrix.android.sdk.internal.util.BackgroundDetectionObserver
 import org.matrix.android.sdk.internal.worker.MatrixWorkerFactory
-import org.matrix.olm.OlmManager
 import java.util.concurrent.Executors
 import javax.inject.Inject
 
@@ -49,7 +48,6 @@ internal class TestMatrix(context: Context, matrixConfiguration: MatrixConfigura
     @Inject internal lateinit var rawService: RawService
     @Inject internal lateinit var userAgentHolder: UserAgentHolder
     @Inject internal lateinit var backgroundDetectionObserver: BackgroundDetectionObserver
-    @Inject internal lateinit var olmManager: OlmManager
     @Inject internal lateinit var sessionManager: SessionManager
     @Inject internal lateinit var homeServerHistoryService: HomeServerHistoryService
     @Inject internal lateinit var apiInterceptor: ApiInterceptor

diff --git a/...roidTest/java/org/matrix/android/sdk/internal/crypto/keysbackup/KeysBackupPasswordTest.kt b/...roidTest/java/org/matrix/android/sdk/internal/crypto/keysbackup/KeysBackupPasswordTest.kt
@@ -20,7 +20,6 @@ import androidx.test.ext.junit.runners.AndroidJUnit4
 import org.junit.Assert.assertArrayEquals
 import org.junit.Assert.assertEquals
 import org.junit.Assert.assertTrue
-import org.junit.Before
 import org.junit.FixMethodOrder
 import org.junit.Ignore
 import org.junit.Test
@@ -29,19 +28,12 @@ import org.junit.runners.MethodSorters
 import org.matrix.android.sdk.InstrumentedTest
 import org.matrix.android.sdk.api.listeners.ProgressListener
 import org.matrix.android.sdk.common.assertByteArrayNotEqual
-import org.matrix.olm.OlmManager
-import org.matrix.olm.OlmPkDecryption
 
 @Ignore("Ignored in order to speed up test run time")
 @RunWith(AndroidJUnit4::class)
 @FixMethodOrder(MethodSorters.JVM)
 class KeysBackupPasswordTest : InstrumentedTest {
 
-    @Before
-    fun ensureLibLoaded() {
-        OlmManager()
-    }
-
     /**
      * Check KeysBackupPassword utilities
      */
@@ -51,7 +43,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
 
         assertEquals(32, generatePrivateKeyResult.salt.length)
         assertEquals(500_000, generatePrivateKeyResult.iterations)
-        assertEquals(OlmPkDecryption.privateKeyLength(), generatePrivateKeyResult.privateKey.size)
+        assertEquals(EXPECTED_PRIVATE_KEY_LENGTH, generatePrivateKeyResult.privateKey.size)
 
         // Reverse operation
         val retrievedPrivateKey = retrievePrivateKeyWithPassword(
@@ -60,7 +52,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
                 generatePrivateKeyResult.iterations
         )
 
-        assertEquals(OlmPkDecryption.privateKeyLength(), retrievedPrivateKey.size)
+        assertEquals(EXPECTED_PRIVATE_KEY_LENGTH, retrievedPrivateKey.size)
         assertArrayEquals(generatePrivateKeyResult.privateKey, retrievedPrivateKey)
     }
 
@@ -101,7 +93,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
 
         assertEquals(32, generatePrivateKeyResult.salt.length)
         assertEquals(500_000, generatePrivateKeyResult.iterations)
-        assertEquals(OlmPkDecryption.privateKeyLength(), generatePrivateKeyResult.privateKey.size)
+        assertEquals(EXPECTED_PRIVATE_KEY_LENGTH, generatePrivateKeyResult.privateKey.size)
 
         // Reverse operation, with bad password
         val retrievedPrivateKey = retrievePrivateKeyWithPassword(
@@ -110,7 +102,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
                 generatePrivateKeyResult.iterations
         )
 
-        assertEquals(OlmPkDecryption.privateKeyLength(), retrievedPrivateKey.size)
+        assertEquals(EXPECTED_PRIVATE_KEY_LENGTH, retrievedPrivateKey.size)
         assertByteArrayNotEqual(generatePrivateKeyResult.privateKey, retrievedPrivateKey)
     }
 
@@ -123,7 +115,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
 
         assertEquals(32, generatePrivateKeyResult.salt.length)
         assertEquals(500_000, generatePrivateKeyResult.iterations)
-        assertEquals(OlmPkDecryption.privateKeyLength(), generatePrivateKeyResult.privateKey.size)
+        assertEquals(EXPECTED_PRIVATE_KEY_LENGTH, generatePrivateKeyResult.privateKey.size)
 
         // Reverse operation, with bad iteration
         val retrievedPrivateKey = retrievePrivateKeyWithPassword(
@@ -132,7 +124,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
                 500_001
         )
 
-        assertEquals(OlmPkDecryption.privateKeyLength(), retrievedPrivateKey.size)
+        assertEquals(EXPECTED_PRIVATE_KEY_LENGTH, retrievedPrivateKey.size)
         assertByteArrayNotEqual(generatePrivateKeyResult.privateKey, retrievedPrivateKey)
     }
 
@@ -145,7 +137,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
 
         assertEquals(32, generatePrivateKeyResult.salt.length)
         assertEquals(500_000, generatePrivateKeyResult.iterations)
-        assertEquals(OlmPkDecryption.privateKeyLength(), generatePrivateKeyResult.privateKey.size)
+        assertEquals(EXPECTED_PRIVATE_KEY_LENGTH, generatePrivateKeyResult.privateKey.size)
 
         // Reverse operation, with bad iteration
         val retrievedPrivateKey = retrievePrivateKeyWithPassword(
@@ -154,7 +146,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
                 generatePrivateKeyResult.iterations
         )
 
-        assertEquals(OlmPkDecryption.privateKeyLength(), retrievedPrivateKey.size)
+        assertEquals(EXPECTED_PRIVATE_KEY_LENGTH, retrievedPrivateKey.size)
         assertByteArrayNotEqual(generatePrivateKeyResult.privateKey, retrievedPrivateKey)
     }
 
@@ -169,7 +161,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
 
         val retrievedPrivateKey = retrievePrivateKeyWithPassword(password, salt, iteration)
 
-        assertEquals(OlmPkDecryption.privateKeyLength(), retrievedPrivateKey.size)
+        assertEquals(EXPECTED_PRIVATE_KEY_LENGTH, retrievedPrivateKey.size)
 
         // Data from RiotWeb
         val privateKeyBytes = byteArrayOf(
@@ -187,5 +179,7 @@ class KeysBackupPasswordTest : InstrumentedTest {
         private const val BAD_PASSWORD = "passw0rd"
 
         private const val BAD_SALT = "AA0lxhQ9aYgGfMsclVWPIAublg8h9Nlu"
+
+        private const val EXPECTED_PRIVATE_KEY_LENGTH = 32
     }
 }
diff --git a/...droid/sdk/internal/crypto/store/migration/DynamicElementAndroidToElementRMigrationTest.kt b/...droid/sdk/internal/crypto/store/migration/DynamicElementAndroidToElementRMigrationTest.kt
diff --git a/...rc/androidTest/java/org/matrix/android/sdk/internal/database/CryptoSanityMigrationTest.kt b/...rc/androidTest/java/org/matrix/android/sdk/internal/database/CryptoSanityMigrationTest.kt