use do catch, more details in error log

RevenueCat · Apr 28, 2022 · 6a7697b · NachoSoto · Apr 28, 2022 · NachoSoto
1 parent a11742a
commit 6a7697b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/Sources/Attribution/AttributionFetcher.swift b/Sources/Attribution/AttributionFetcher.swift
@@ -77,10 +77,12 @@ class AttributionFetcher {
     @available(iOS 14.3, macOS 11.1, macCatalyst 14.3, *)
     var adServicesToken: String? {
 #if canImport(AdServices)
-        if let attributionToken = try? AAAttribution.attributionToken() {
+        do {
+            let attributionToken = try AAAttribution.attributionToken()
             return attributionToken
-        } else {
-            Logger.warn(Strings.attribution.adservices_token_fetch_failed)
+        } catch {
+            let message = Strings.attribution.adservices_token_fetch_failed(error: error)
+            Logger.appleWarning(message)
             return nil
         }
 #else

diff --git a/Sources/Logging/Strings/AttributionStrings.swift b/Sources/Logging/Strings/AttributionStrings.swift
@@ -39,7 +39,7 @@ enum AttributionStrings {
     case attribute_set_locally(attribute: String)
     case missing_advertiser_identifiers
     case adservices_not_supported
-    case adservices_token_fetch_failed
+    case adservices_token_fetch_failed(error: Error?)
 
 }
 
@@ -120,8 +120,8 @@ extension AttributionStrings: CustomStringConvertible {
             return "Tried to fetch AdServices attribution token on device without " +
                 "AdServices support."
 
-        case .adservices_token_fetch_failed:
-            return "Error fetching AdServices attribution token."
+        case .adservices_token_fetch_failed(let error):
+            return "Fetching AdServices attribution token failed with error: \(error?.localizedDescription ?? "")"
         }
     }