feat: remove KVStorage "state" singleton, and replace it with ad-hoc …

…KVStorage instances (#116)
immuni-app · May 29, 2020 · d866417 · d866417
1 parent 3cc9f7f
commit d866417
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 13 deletions.
diff --git a/app/src/main/java/it/ministerodellasalute/immuni/koinModule.kt b/app/src/main/java/it/ministerodellasalute/immuni/koinModule.kt
@@ -66,12 +66,6 @@ import org.koin.dsl.module
  * Dependency Injection Koin module.
  */
 val appModule = module {
-
-    /**
-     * KVStorage to store generic non-database data encrypted using AES256.
-     */
-    single { KVStorage("state", androidContext(), encrypted = true, moshi = get()) }
-
     /**
      * App Configuration Service APIs
      */
@@ -132,7 +126,12 @@ val appModule = module {
 
     single {
         ConfigurationSettingsStoreRepository(
-            get(),
+            KVStorage(
+                name = "ConfigurationSettingsStoreRepository",
+                context = androidContext(),
+                encrypted = true,
+                moshi = get()
+            ),
             defaultSettings
         )
     }
@@ -174,7 +173,14 @@ val appModule = module {
     }
 
     single {
-        UploadDisablerStore(get())
+        UploadDisablerStore(
+            KVStorage(
+                name = "UploadDisablerStore",
+                context = androidContext(),
+                encrypted = true,
+                moshi = get()
+            )
+        )
     }
 
     single {
@@ -229,7 +235,15 @@ val appModule = module {
     }
 
     single {
-        UserRepository(get())
+        UserRepository(
+            KVStorage(
+                name = "UserRepository",
+                context = androidContext(),
+                encrypted = true,
+                moshi = immuniMoshi
+            )
+
+        )
     }
 
     single {

diff --git a/extensions/src/main/java/it/ministerodellasalute/immuni/extensions/storage/KVStorage.kt b/extensions/src/main/java/it/ministerodellasalute/immuni/extensions/storage/KVStorage.kt
@@ -52,7 +52,7 @@ class KVStorage(
     val name: String,
     context: Context?,
     val cacheInMemory: Boolean = true,
-    val encrypted: Boolean,
+    val encrypted: Boolean = true,
     val moshi: Moshi,
     // The following two properties should be private but are not, because they're used in public
     // inline methods, thus the underscore prefix.
@@ -67,7 +67,7 @@ class KVStorage(
         mutableMapOf()
 
     /**
-     * Checks if the storage contains the given key.
+     * Checks if the storage contains the given [key].
      */
     fun <T : Any> contains(key: Key<T>): Boolean = synchronized(this) {
         _cache.contains(key) || _sharedPrefs.contains(key.name)
@@ -85,8 +85,8 @@ class KVStorage(
     }
 
     /**
-     * Saves value for the given key. If value's type is not one of the supported primitive types
-     * (Boolean, Int, Long, Float, String), it first serializes value to Json with Moshi
+     * Saves [value] for the given [key]. If value's type is not one of the supported primitive types
+     * (Boolean, Int, Long, Float, String), it first serializes value to Json with [Moshi]
      * and then saves it as a String.
      */
     inline operator fun <reified T : Any> set(key: Key<T>, value: T) = synchronized(this) {