@verdocs/js-sdk 4.2.74 → 4.2.75

package/dist/index.js

@@ -1287,27 +1287,26 @@ class VerdocsEndpoint {
         if (this.persist) {
             localStorage.setItem(this.sessionStorageKey(), token);
         }
-        if (!session.sub || this.sub !== session.sub) {
-            // The main purpose for this var is to prevent dupe getCurrentProfile calls and the trigger for the dupe
-            // is usually two components calling loadSession() at the same time to ensure we've done all we could
-            // to load it. We expose it in case it's useful, but have to set it here instead of in the async callbacks
-            // even though their might be a brief window of time where it's set but something might go wrong (profile
-            // doesn't load) because we need it to serve that original purpose more.
-            window?.console?.debug('[JS_SDK] userId changed, loading current profile...', this.endpointId, this.sub, session.sub);
-            this.sub = session.sub;
-            getCurrentProfile(this)
-                .then((r) => {
-                window?.console?.debug('[JS_SDK] Loaded profile', this.endpointId, r);
-                this.profile = r || null;
-                this.notifySessionListeners();
-            })
-                .catch((e) => {
-                this.profile = null;
-                this.sub = null;
-                window?.console?.warn('Unable to load profile', e);
-                this.notifySessionListeners();
-            });
-        }
+        // NOTE: We don't attempt to de-dupe session/network operations because there are cases where we
+        // operate within an IFRAME and multiple siblings need to operate on the same session. This happens
+        // a lot in Storybook but may also happen in other environments as well. Rather than try to coordinate
+        // something weird that might break (e.g. in a mobile app), we just allow the dupe calls. We generate
+        // a unique ID for each endpoint just to help with debugging. It should not be taken as a bug if more
+        // than one endpoint ID is seen within the app's logs.
+        window?.console?.debug('[JS_SDK] Loading current profile...', this.endpointId, this.sub, session.sub);
+        this.sub = session.sub;
+        getCurrentProfile(this)
+            .then((r) => {
+            window?.console?.debug('[JS_SDK] Loaded profile', this.endpointId, r);
+            this.profile = r || null;
+            this.notifySessionListeners();
+        })
+            .catch((e) => {
+            this.profile = null;
+            this.sub = null;
+            window?.console?.warn('Unable to load profile', e);
+            this.notifySessionListeners();
+        });
         return this;
     }
     /**