Update calls to AutofillCredentialIdentityStoreManager to optionally …

…provide a vault (and always provide SecureVaultReporter() since AutofillCredentialIdentityStoreManager can now init the vault lazily)
duckduckgo · Jan 3, 2025 · f8d2dc8 · f8d2dc8
1 parent b84cea1
commit f8d2dc8
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 21 deletions.
diff --git a/AutofillCredentialProvider/CredentialProvider/CredentialProviderViewController.swift b/AutofillCredentialProvider/CredentialProvider/CredentialProviderViewController.swift
@@ -33,9 +33,11 @@ class CredentialProviderViewController: ASCredentialProviderViewController {
     private lazy var authenticator = UserAuthenticator(reason: UserText.credentialProviderListAuthenticationReason,
                                                        cancelTitle: UserText.credentialProviderListAuthenticationCancelButton)
 
-    private lazy var credentialIdentityStoreManager: AutofillCredentialIdentityStoreManaging = AutofillCredentialIdentityStoreManager(credentialStore: ASCredentialIdentityStore.shared,
-                                                                                                                                      vault: secureVault,
-                                                                                                                                      tld: tld)
+    private lazy var credentialIdentityStoreManager: AutofillCredentialIdentityStoreManaging = AutofillCredentialIdentityStoreManager(
+        credentialStore: ASCredentialIdentityStore.shared,
+        vault: secureVault,
+        reporter: SecureVaultReporter(),
+        tld: tld)
 
     private lazy var secureVault: (any AutofillSecureVault)? = {
         if findKeychainItemsWithV4() {

diff --git a/Core/SyncCredentialsAdapter.swift b/Core/SyncCredentialsAdapter.swift
@@ -47,8 +47,7 @@ public final class SyncCredentialsAdapter {
             secureVaultErrorReporter: secureVaultErrorReporter,
             errorEvents: CredentialsCleanupErrorHandling()
         )
-        credentialIdentityStoreManager = AutofillCredentialIdentityStoreManager(
-            vault: try? secureVaultFactory.makeVault(reporter: secureVaultErrorReporter), tld: tld)
+        credentialIdentityStoreManager = AutofillCredentialIdentityStoreManager(reporter: secureVaultErrorReporter, tld: tld)
     }
 
     public func cleanUpDatabaseAndUpdateSchedule(shouldEnable: Bool) {

diff --git a/DuckDuckGo/AutofillLoginListViewModel.swift b/DuckDuckGo/AutofillLoginListViewModel.swift
@@ -88,7 +88,9 @@ final class AutofillLoginListViewModel: ObservableObject {
         return settings["monitorIntervalDays"] as? Int ?? 42
     }()
 
-    private lazy var credentialIdentityStoreManager: AutofillCredentialIdentityStoreManaging = AutofillCredentialIdentityStoreManager(vault: secureVault, tld: tld)
+    private lazy var credentialIdentityStoreManager: AutofillCredentialIdentityStoreManaging = AutofillCredentialIdentityStoreManager(vault: secureVault,
+                                                                                                                                      reporter: SecureVaultReporter(),
+                                                                                                                                      tld: tld)
 
     private lazy var syncPromoManager: SyncPromoManaging = SyncPromoManager(syncService: syncService)
 

diff --git a/DuckDuckGo/AutofillUsageMonitor.swift b/DuckDuckGo/AutofillUsageMonitor.swift
@@ -23,12 +23,8 @@ import BrowserServicesKit
 
 final class AutofillUsageMonitor {
 
-    private lazy var credentialIdentityStoreManager: AutofillCredentialIdentityStoreManager? = {
-        guard let vault = try? AutofillSecureVaultFactory.makeVault(reporter: SecureVaultReporter()) else {
-            return nil
-        }
-
-        return AutofillCredentialIdentityStoreManager(vault: vault,
+    private lazy var credentialIdentityStoreManager: AutofillCredentialIdentityStoreManager = {
+        return AutofillCredentialIdentityStoreManager(reporter: SecureVaultReporter(),
                                                       tld: AppDependencyProvider.shared.storageCache.tld)
     }()
 
@@ -39,7 +35,7 @@ final class AutofillUsageMonitor {
             if state.isEnabled {
                 if self?.autofillExtensionEnabled == nil {
                     Task {
-                        await self?.credentialIdentityStoreManager?.populateCredentialStore()
+                        await self?.credentialIdentityStoreManager.populateCredentialStore()
                     }
                 }
                 self?.autofillExtensionEnabled = true

diff --git a/DuckDuckGo/TabViewController.swift b/DuckDuckGo/TabViewController.swift
@@ -276,12 +276,8 @@ class TabViewController: UIViewController {
         return manager
     }()
 
-    private lazy var credentialIdentityStoreManager: AutofillCredentialIdentityStoreManager? = {
-        guard let vault = try? AutofillSecureVaultFactory.makeVault(reporter: SecureVaultReporter()) else {
-            return nil
-        }
-
-        return AutofillCredentialIdentityStoreManager(vault: vault,
+    private lazy var credentialIdentityStoreManager: AutofillCredentialIdentityStoreManager = {
+        return AutofillCredentialIdentityStoreManager(reporter: SecureVaultReporter(),
                                                       tld: AppDependencyProvider.shared.storageCache.tld)
     }()
 
@@ -2851,7 +2847,7 @@ extension TabViewController: SecureVaultManagerDelegate {
 
                 guard let domain = account?.domain else { return }
                 Task {
-                    await self?.credentialIdentityStoreManager?.updateCredentialStore(for: domain)
+                    await self?.credentialIdentityStoreManager.updateCredentialStore(for: domain)
                 }
             } completionHandler: { account in
                 if account != nil {
@@ -3049,7 +3045,7 @@ extension TabViewController: SaveLoginViewControllerDelegate {
 
                 guard let domain = newCredential.account.domain else { return }
                 Task {
-                    await credentialIdentityStoreManager?.updateCredentialStore(for: domain)
+                    await credentialIdentityStoreManager.updateCredentialStore(for: domain)
                 }
             }
         } catch {