Improved logging around push notifications

wultra · Nov 20, 2024 · 6fb1e9c · 6fb1e9c
1 parent 7ee8a01
commit 6fb1e9c
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 4 deletions.
diff --git a/WultraMobileTokenSDK/Common/WMTProvisioningUtils.swift b/WultraMobileTokenSDK/Common/WMTProvisioningUtils.swift
@@ -19,8 +19,9 @@ import Foundation
 class WMTProvisioningUtils {
 
     static func getMainProvisioningProfile() -> WMTProvision? {
+        D.debug("Retrieving embedded provisioning profile from the main bundle.")
         guard let filePath = Bundle.main.path(forResource: "embedded", ofType: "mobileprovision") else {
-            D.debug("Missing embedded provisioning profile in the main bundle.")
+            D.error("Missing embedded provisioning profile in the main bundle.")
             return nil
         }
         let url = URL(fileURLWithPath: filePath)
@@ -60,6 +61,7 @@ class WMTProvisioningUtils {
     static func parseProvisioningProfilePlist(_ plist: Data) -> WMTProvision? {
         do {
             let provision = try PropertyListDecoder().decode(WMTProvision.self, from: plist)
+            D.debug("Successfully parsed provisioning profile (apns env: \(provision.entitlements.apsEnvironment?.rawValue ?? "nil")).")
             return provision
         } catch let e {
             D.error("Failed to parse provisioning profile: \(e)")

diff --git a/WultraMobileTokenSDK/Push/Service/WMTPushImpl.swift b/WultraMobileTokenSDK/Push/Service/WMTPushImpl.swift
@@ -87,6 +87,8 @@ class WMTPushImpl: WMTPush, WMTService {
             payloadEnvironment = nil // no env for FCM
         }
 
+        D.info("Registering push for \(payloadPlatform.rawValue) platform.")
+
         return registerPush(platform: payloadPlatform, token: payloadToken, environment: payloadEnvironment, completion: completion)
     }
 
@@ -122,9 +124,20 @@ class WMTPushImpl: WMTPush, WMTService {
 
     private func getPushEnvironment(environment: WMTPushAPNSEnvironment) -> WMTPushRegistrationEnvironment? {
         switch environment {
-        case .development: return .development
-        case .production: return .production
-        case .automatic: return WMTProvisioningUtils.getMainProvisioningProfile()?.apnsEnvironment
+        case .development:
+            D.info("Using APNS development environment for push notifications.")
+            return .development
+        case .production:
+            D.info("Using APNS production environment for push notifications.")
+            return .production
+        case .automatic:
+            let env = WMTProvisioningUtils.getMainProvisioningProfile()?.apnsEnvironment
+            if let env {
+                D.info("Using \(env) environment for push notifications (automatic resolution).")
+            } else {
+                D.warning("No APNS environment found in provisioning profile. Server configuration will be used.")
+            }
+            return env
         }
     }
 }