fix: only show production warning when explicity setting the 'useInsecureForRefreshToken'

DanielRivers · DanielRivers · commit 958f4edbca16 · 2025-01-20T19:59:48.000Z
diff --git a/lib/sessionManager/stores/localStorage.ts b/lib/sessionManager/stores/localStorage.ts
@@ -12,7 +12,9 @@ export class LocalStorage<V extends string = StorageKeys>
 {
   constructor() {
     super();
-    console.warn("LocalStorage store should not be used in production");
+    if (storageSettings.useInsecureForRefreshToken) {
+      console.warn("LocalStorage store should not be used in production");
+    }
   }
 
   private internalItems: Set<V | StorageKeys> = new Set<V>();

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,9 @@ export class LocalStorage<V extends string = StorageKeys>`
`12`	`12`	`{`
`13`	`13`	`constructor() {`
`14`	`14`	`super();`
`15`		`- console.warn("LocalStorage store should not be used in production");`
	`15`	`+ if (storageSettings.useInsecureForRefreshToken) {`
	`16`	`+ console.warn("LocalStorage store should not be used in production");`
	`17`	`+ }`
`16`	`18`	`}`
`17`	`19`
`18`	`20`	`private internalItems: Set<V \| StorageKeys> = new Set<V>();`