fix(sdk): safe access localstorage in browser detection

Author: ScriptedAlchemy

.changeset/tall-balloons-clean.md

@@ -0,0 +1,5 @@
+---
+'@module-federation/sdk': patch
+---
+
+Resolving issue where localstorage is disabled, for cases like android WebView, where it will be null. This wraps localstorage checks in try catch will fallback to see if document is not undefined

packages/sdk/src/logger.ts

@@ -10,15 +10,23 @@ function safeToString(info: any): string {
 }
 
 const DEBUG_LOG = '[ FEDERATION DEBUG ]';
+
+function safeGetLocalStorageItem() {
+  try {
+    if (typeof window !== 'undefined' && window.localStorage) {
+      return localStorage.getItem(BROWSER_LOG_KEY) === BROWSER_LOG_VALUE;
+    }
+  } catch (error) {
+    return typeof typeof document !== 'undefined';
+  }
+  return false;
+}
 class Logger {
   enable = false;
   identifier: string;
   constructor(identifier?: string) {
     this.identifier = identifier || DEBUG_LOG;
-    if (
-      isBrowserEnv() &&
-      localStorage.getItem(BROWSER_LOG_KEY) === BROWSER_LOG_VALUE
-    ) {
+    if (isBrowserEnv() && safeGetLocalStorageItem()) {
       this.enable = true;
     } else if (isDebugMode()) {
       this.enable = true;