fix(oidc): json not valid add warning message (release) (#1258)

AxaFrance · Feb 3, 2024 · 940ffac · 940ffac
1 parent 31f295c
commit 940ffac
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/packages/oidc-client/src/initSession.ts b/packages/oidc-client/src/initSession.ts
@@ -64,6 +64,12 @@ export const initSession = (configurationName, storage = sessionStorage) => {
     };
     const getLoginParams = () => {
         const dataString = storage[`oidc.login.${configurationName}`];
+
+        if(!dataString){
+            console.warn(`storage[oidc.login.${configurationName}] is empty`);
+            return null;
+        }
+
         if (!getLoginParamsCache[configurationName]) {
             getLoginParamsCache[configurationName] = JSON.parse(dataString);
         }

diff --git a/packages/oidc-client/src/oidc.ts b/packages/oidc-client/src/oidc.ts
@@ -234,8 +234,7 @@ Please checkout that you are using OIDC hook inside a <OidcProvider configuratio
     async tryKeepExistingSessionAsync() :Promise<boolean> {
         if (this.tryKeepExistingSessionPromise !== null) {
             return this.tryKeepExistingSessionPromise;
-        }
-
+        }        
         this.tryKeepExistingSessionPromise = tryKeepSessionAsync(this);
         return this.tryKeepExistingSessionPromise.then((result) => {
             this.tryKeepExistingSessionPromise = null;
@@ -311,7 +310,7 @@ Please checkout that you are using OIDC hook inside a <OidcProvider configuratio
         }
         const configuration = this.configuration;
 
-        const silentLoginAsync = (extras: StringMap, state:string, scope:string = null) => {
+        const silentLoginAsync = (extras: StringMap, state:string=null, scope:string = null) => {
             return _silentLoginAsync(this.configurationName, this.configuration, this.publishEvent.bind(this))(extras, state, scope);
         };
         const localsilentLoginAsync = async () => {
@@ -328,7 +327,7 @@ Please checkout that you are using OIDC hook inside a <OidcProvider configuratio
                     ...loginParams.extras,
                     ...extras,
                     prompt: 'none',
-                }, loginParams.state);
+                });
                 if (silent_token_response) {
                     updateTokens(silent_token_response.tokens);
                     this.publishEvent(Oidc.eventNames.token_renewed, {});