@explorins/pers-signer 1.0.31 → 1.0.33

package/dist/index.esm.js

@@ -392,7 +392,7 @@ class AuthenticationService {
     async combinedAuthentication(identifier, persAccessToken) {
         try {
             // Step 1: Try to login with PERS token first (this will get signer JWT if user exists)
-            let signerToken;
+            let signerToken = null;
             try {
                 const loginResult = await this.loginWithPersToken(persAccessToken);
                 // Extract token from login result
@@ -406,6 +406,11 @@ class AuthenticationService {
             catch (loginError) {
                 // Step 2: User doesn't exist - register with v1 API (Unified Flow)
                 try {
+                    // Only proceed to registration if the error is explicitly 'user_not_found'
+                    const errorMessage = loginError instanceof Error ? loginError.message : String(loginError);
+                    if (errorMessage !== 'user_not_found') {
+                        throw loginError;
+                    }
                     const registerResult = await this.registerUser(persAccessToken);
                     if (registerResult && registerResult.access_token) {
                         signerToken = registerResult.access_token;
@@ -416,9 +421,13 @@ class AuthenticationService {
                 }
                 catch (registerError) {
                     console.error(`[AuthenticationService] Registration failed for ${identifier}:`, registerError);
+                    // Explicitly rethrow to abort the entire authentication process
                     throw registerError;
                 }
             }
+            if (!signerToken) {
+                throw new Error('Authentication failed: Unable to obtain signer token');
+            }
             const user = {
                 identifier: identifier,
                 signerAuthToken: signerToken,
@@ -428,8 +437,9 @@ class AuthenticationService {
             return user;
         }
         catch (error) {
-            console.error(`[PersSignerSDK] Combined authentication failed for ${identifier}:`, error);
-            throw new Error(`Combined authentication failed: ${error}`);
+            const errorMessage = error instanceof Error ? error.message : String(error);
+            console.error(`[PersSignerSDK] Combined authentication failed for ${identifier}:`, errorMessage);
+            throw new Error(`Combined authentication failed: ${errorMessage}`);
         }
     }
 }
@@ -2147,12 +2157,16 @@ class PersSignerSDK {
      */
     constructor(config) {
         this.config = config;
+        // Safe hostname/origin extraction - window.location may be undefined in React Native
+        const hasWindowLocation = typeof window !== 'undefined' && typeof window.location !== 'undefined';
+        const hostname = hasWindowLocation ? window.location.hostname : 'localhost';
+        const origin = hasWindowLocation ? window.location.origin : undefined;
         setConfigProvider(new WebConfigProvider({
             apiUrl: config.apiUrl || SIGNER_CONFIG.DEFAULT_SIGNER_API_URL,
             relyingParty: {
-                id: typeof window !== 'undefined' ? window.location.hostname : 'localhost',
+                id: hostname,
                 name: config.relyingPartyName || SIGNER_CONFIG.DEFAULT_RELYING_PARTY_NAME,
-                origin: typeof window !== 'undefined' ? window.location.origin : undefined,
+                origin: origin,
             },
         }));
         this.authenticationService = new AuthenticationService(this.config);