fix: server app should initialize in web workers

firebase · Jun 27, 2024 · 3a2a1ba · 3a2a1ba
1 parent baa96ce
commit 3a2a1ba
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/.changeset/fifty-mayflies-hug.md b/.changeset/fifty-mayflies-hug.md
@@ -0,0 +1,5 @@
+---
+'@firebase/app': patch
+---
+
+fix: server app should initialize in web workers
diff --git a/packages/app/src/api.ts b/packages/app/src/api.ts
@@ -48,7 +48,12 @@ import {
   LogOptions,
   setUserLogHandler
 } from '@firebase/logger';
-import { deepEqual, getDefaultAppConfig, isBrowser } from '@firebase/util';
+import {
+  deepEqual,
+  getDefaultAppConfig,
+  isBrowser,
+  isWebWorker
+} from '@firebase/util';
 
 export { FirebaseError } from '@firebase/util';
 
@@ -230,7 +235,7 @@ export function initializeServerApp(
   _options: FirebaseOptions | FirebaseApp,
   _serverAppConfig: FirebaseServerAppSettings
 ): FirebaseServerApp {
-  if (isBrowser()) {
+  if (isBrowser() && !isWebWorker()) {
     // FirebaseServerApp isn't designed to be run in browsers.
     throw ERROR_FACTORY.create(AppError.INVALID_SERVER_APP_ENVIRONMENT);
   }