@superfan-app/spotify-auth 0.1.81 → 0.1.83

package/ios/SpotifyASWebAuthSession.swift

@@ -10,9 +10,7 @@ final class SpotifyASWebAuthSession {
     private var isAuthenticating = false
     
     private func secureLog(_ message: String) {
-        #if DEBUG
-        print("[SpotifyASWebAuth] \(message)")
-        #endif
+        NSLog("[SpotifyASWebAuth] \(message)")
     }
     
     func startAuthFlow(
@@ -22,12 +20,14 @@ final class SpotifyASWebAuthSession {
         completion: @escaping (Result<URL, Error>) -> Void
     ) {
         guard !isAuthenticating else {
+            secureLog("Auth already in progress, ignoring duplicate start")
             completion(.failure(SpotifyAuthError.authorizationError("Authentication already in progress")))
             return
         }
         
         isAuthenticating = true
-        
+        secureLog("Starting auth flow for URL: \(authUrl.absoluteString)")
+
         // Create the auth session
         authSession = ASWebAuthenticationSession(
             url: authUrl,
@@ -37,17 +37,22 @@ final class SpotifyASWebAuthSession {
             
             if let error = error {
                 if (error as NSError).code == ASWebAuthenticationSessionError.canceledLogin.rawValue {
+                    self?.secureLog("User cancelled auth session")
                     completion(.failure(SpotifyAuthError.userCancelled))
                 } else {
+                    self?.secureLog("Auth session error: \(error.localizedDescription)")
                     completion(.failure(SpotifyAuthError.networkError(error.localizedDescription)))
                 }
                 return
             }
-            
+
             guard let callbackURL = callbackURL else {
+                self?.secureLog("Auth session returned no callback URL")
                 completion(.failure(SpotifyAuthError.authorizationError("No callback URL received")))
                 return
             }
+
+            self?.secureLog("Auth session callback received: \(callbackURL.absoluteString)")
             
             completion(.success(callbackURL))
         }
@@ -62,9 +67,10 @@ final class SpotifyASWebAuthSession {
         
         // Start the session
         DispatchQueue.main.async { [weak self] in
-            guard let self = self, 
+            guard let self = self,
                   let authSession = self.authSession,
                   authSession.start() else {
+                self?.secureLog("Failed to start ASWebAuthenticationSession")
                 self?.isAuthenticating = false
                 completion(.failure(SpotifyAuthError.authorizationError("Failed to start auth session")))
                 return

package/ios/SpotifyAuthAuth.swift

@@ -275,7 +275,7 @@ final class SpotifyAuthAuth: NSObject, SPTSessionManagerDelegate {
           .accessibility(.afterFirstUnlock)
         try keychain.set(refreshToken, key: keychainKey)
       } catch {
-        print("Failed to store refresh token securely: \(error)")
+        NSLog("[SpotifyAuth] Failed to store refresh token securely: \(error)")
       }
     }
   }
@@ -288,7 +288,7 @@ final class SpotifyAuthAuth: NSObject, SPTSessionManagerDelegate {
       let keychain = Keychain(service: Bundle.main.bundleIdentifier ?? "com.superfan.app")
       try keychain.remove(keychainKey)
     } catch {
-      print("Failed to clear previous refresh token: \(error)")
+      NSLog("[SpotifyAuth] Failed to clear previous refresh token: \(error)")
     }
   }
   
@@ -354,32 +354,35 @@ final class SpotifyAuthAuth: NSObject, SPTSessionManagerDelegate {
   
   public func initAuth(config: AuthorizeConfig) {
     do {
+        secureLog("initAuth called, showDialog=\(config.showDialog), campaign=\(config.campaign ?? "nil")")
         guard let sessionManager = self.sessionManager else {
             throw SpotifyAuthError.sessionError("Session manager not initialized")
         }
-        
+
         // Get scopes from Info.plist and convert to SPTScope
         let scopes = try self.scopes
+        secureLog("Scopes from Info.plist: \(scopes.joined(separator: ", "))")
         let sptScopes = scopes.reduce(into: SPTScope()) { result, scopeString in
             if let scope = stringToScope(scopeString: scopeString) {
                 result.insert(scope)
             }
         }
-        
+
         if sptScopes.isEmpty {
             throw SpotifyAuthError.invalidConfiguration("No valid scopes found in configuration")
         }
-        
+
         isAuthenticating = true
-        
+
         if sessionManager.isSpotifyAppInstalled {
+            secureLog("Spotify app is installed, using app-switch flow")
             // Use the native app‑switch flow
             if config.showDialog {
                 sessionManager.alwaysShowAuthorizationDialog = true
             }
             sessionManager.initiateSession(with: sptScopes, options: .default, campaign: config.campaign)
         } else {
-            // Use web auth as fallback
+            secureLog("Spotify app not installed, using web auth fallback")
             // Get configuration from Info.plist before dispatching to main thread
             let clientId = try self.clientID
             let redirectUrl = try self.redirectURL
@@ -492,23 +495,23 @@ final class SpotifyAuthAuth: NSObject, SPTSessionManagerDelegate {
     // URLComponents.percentEncodedQuery properly encodes the values
     request.httpBody = components.percentEncodedQuery?.data(using: .utf8)
     
-    print("[SpotifyDebug] exchangeCodeForToken: sending request to \(url)")
+    NSLog("[SpotifyAuth] exchangeCodeForToken: sending request to \(url)")
     let task = URLSession.shared.dataTask(with: request) { [weak self] data, response, error in
-        print("[SpotifyDebug] exchangeCodeForToken: dataTask callback fired, self=\(self != nil), error=\(String(describing: error))")
+        NSLog("[SpotifyAuth] exchangeCodeForToken: dataTask callback fired, self=\(self != nil), error=\(String(describing: error))")
         if let http = response as? HTTPURLResponse {
-            print("[SpotifyDebug] exchangeCodeForToken: HTTP status \(http.statusCode)")
+            NSLog("[SpotifyAuth] exchangeCodeForToken: HTTP status \(http.statusCode)")
         }
         if let data = data, let body = String(data: data, encoding: .utf8) {
-            print("[SpotifyDebug] exchangeCodeForToken: response body = \(body)")
+            NSLog("[SpotifyAuth] exchangeCodeForToken: response body = \(body)")
         }
         do {
             let responseData = try self?.validateHTTPResponse(data: data, response: response, error: error)
             guard let responseData = responseData else {
-                print("[SpotifyDebug] exchangeCodeForToken: responseData is nil (self may be nil)")
+                NSLog("[SpotifyAuth] exchangeCodeForToken: responseData is nil (self may be nil)")
                 return
             }
             let parsed = try Self.parseTokenJSON(from: responseData)
-            print("[SpotifyDebug] exchangeCodeForToken: parsed token OK, hasRefreshToken=\(parsed.refreshToken != nil)")
+            NSLog("[SpotifyAuth] exchangeCodeForToken: parsed token OK, hasRefreshToken=\(parsed.refreshToken != nil)")
 
             guard let refreshToken = parsed.refreshToken else {
                 throw SpotifyAuthError.tokenError("Missing refresh_token in response")
@@ -517,11 +520,11 @@ final class SpotifyAuthAuth: NSObject, SPTSessionManagerDelegate {
             let expirationDate = Date(timeIntervalSinceNow: parsed.expiresIn)
             let sessionData = SpotifySessionData(accessToken: parsed.accessToken, refreshToken: refreshToken, expirationDate: expirationDate, scope: parsed.scope)
             DispatchQueue.main.async {
-                print("[SpotifyDebug] exchangeCodeForToken: setting currentSession on main thread")
+                NSLog("[SpotifyAuth] exchangeCodeForToken: setting currentSession on main thread")
                 self?.currentSession = sessionData
             }
         } catch {
-            print("[SpotifyDebug] exchangeCodeForToken: caught error: \(error)")
+            NSLog("[SpotifyAuth] exchangeCodeForToken: caught error: \(error)")
             self?.handleError(error, context: "token_exchange")
         }
     }
@@ -702,19 +705,13 @@ final class SpotifyAuthAuth: NSObject, SPTSessionManagerDelegate {
   }
   
   // MARK: - Logging
-  
+
   private func secureLog(_ message: String, sensitive: Bool = false) {
-    #if DEBUG
     if sensitive {
-      print("[SpotifyAuth] ********")
+      NSLog("[SpotifyAuth] ********")
     } else {
-      print("[SpotifyAuth] \(message)")
-    }
-    #else
-    if !sensitive {
-      print("[SpotifyAuth] \(message)")
+      NSLog("[SpotifyAuth] \(message)")
     }
-    #endif
   }
   
   deinit {

package/ios/SpotifyAuthModule.swift

@@ -8,15 +8,15 @@ let spotifyAuthorizationEventName = "onSpotifyAuth"
 #if DEBUG
 func secureLog(_ message: String, sensitive: Bool = false) {
     if sensitive {
-        print("[SpotifyAuth] ********")
+        NSLog("[SpotifyAuth] ********")
     } else {
-        print("[SpotifyAuth] \(message)")
+        NSLog("[SpotifyAuth] \(message)")
     }
 }
 #else
 func secureLog(_ message: String, sensitive: Bool = false) {
     if !sensitive {
-        print("[SpotifyAuth] \(message)")
+        NSLog("[SpotifyAuth] \(message)")
     }
 }
 #endif
@@ -79,7 +79,9 @@ public class SpotifyAuthModule: Module {
 
         // Update the dismissal function
         AsyncFunction("dismissAuthSession") {
-            self.spotifyAuth.cancelWebAuth()
+            DispatchQueue.main.async {
+                self.spotifyAuth.cancelWebAuth()
+            }
         }
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@superfan-app/spotify-auth",
-  "version": "0.1.81",
+  "version": "0.1.83",
   "description": "Spotify OAuth module for Expo",
   "main": "src/index.tsx",
   "types": "build/index.d.ts",