@verdocs/js-sdk 4.2.74 → 4.2.76

package/dist/index.js

@@ -1111,9 +1111,7 @@ class VerdocsEndpoint {
     static getDefault() {
         if (!globalThis$1[ENDPOINT_KEY]) {
             globalThis$1[ENDPOINT_KEY] = new VerdocsEndpoint();
-            // window.console.debug('[JS_SDK] Created default endpoint', globalThis[ENDPOINT_KEY].baseURL);
         }
-        window?.console?.debug('[JS_SDK] Getting default endpoint', globalThis$1[ENDPOINT_KEY].endpointId);
         return globalThis$1[ENDPOINT_KEY];
     }
     /**
@@ -1287,27 +1285,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;
     }
     /**
@@ -1371,6 +1368,9 @@ class VerdocsEndpoint {
         this.nextListenerId++;
         const listenerSymbol = Symbol.for('' + this.nextListenerId);
         this.sessionListeners.set(listenerSymbol, listener);
+        // Perform an immediate notification in case this listener subscribed after the endpoint's session was
+        // already loaded.
+        listener(this, this.session, this.profile);
         return () => {
             this.sessionListeners.delete(listenerSymbol);
         };