DRY clearing sku creds.

brave · Nov 18, 2022 · c700c1c · c700c1c
1 parent 11b1ef3
commit c700c1c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/Client/BraveSkus/BraveSkusManager.swift b/Client/BraveSkus/BraveSkusManager.swift
@@ -42,9 +42,7 @@ public class BraveSkusManager {
 
     Logger.module.debug("Refreshing sku credential. Clearing old credential from persistence.")
 
-    Preferences.VPN.skusCredential.reset()
-    Preferences.VPN.skusCredentialDomain.reset()
-    Preferences.VPN.skusCredentialExpirationDate.reset()
+    BraveVPN.clearSkusCredentials()
 
     manager.credentialSummary(for: domain) { completion in
       Logger.module.debug("credentialSummary response")

diff --git a/Sources/BraveVPN/BraveVPN.swift b/Sources/BraveVPN/BraveVPN.swift
@@ -307,6 +307,10 @@ public class BraveVPN {
     GRDKeychain.removeGuardianKeychainItems()
     GRDKeychain.removeSubscriberCredential(withRetries: 3)
 
+    clearSkusCredentials() 
+  }
+
+  public static func clearSkusCredentials() {
     Preferences.VPN.skusCredential.reset()
     Preferences.VPN.skusCredentialDomain.reset()
     Preferences.VPN.skusCredentialExpirationDate.reset()

diff --git a/Sources/BraveVPN/IAPObserver.swift b/Sources/BraveVPN/IAPObserver.swift
@@ -60,9 +60,7 @@ public class IAPObserver: NSObject, SKPaymentTransactionObserver {
               //
               // The user will be able to retrieve the shared credential
               // after log in to account.brave website.
-              Preferences.VPN.skusCredential.reset()
-              Preferences.VPN.skusCredentialDomain.reset()
-              Preferences.VPN.skusCredentialExpirationDate.reset()
+              BraveVPN.clearSkusCredentials()
             } else {
               // Receipt either expired or receipt validation returned some error.
               self.delegate?.purchaseFailed(error: .receiptError)