From 29d034072c20af394ce384e42aa10a37d5dfcb18 Mon Sep 17 00:00:00 2001 From: Christina Holland Date: Fri, 7 Oct 2022 15:07:42 -0700 Subject: [PATCH] Remove json path and fix process check (#6665) --- .changeset/strong-squids-dress.md | 5 +++++ packages/util/src/defaults.ts | 22 +--------------------- 2 files changed, 6 insertions(+), 21 deletions(-) create mode 100644 .changeset/strong-squids-dress.md diff --git a/.changeset/strong-squids-dress.md b/.changeset/strong-squids-dress.md new file mode 100644 index 00000000000..dc68a002421 --- /dev/null +++ b/.changeset/strong-squids-dress.md @@ -0,0 +1,5 @@ +--- +'@firebase/util': patch +--- + +Remove `__FIREBASE_DEFAULTS_PATH__` option for now, as the current implementation causes Webpack warnings. Also fix `process.env` check to work in environments where `process` exists but `process.env` does not. diff --git a/packages/util/src/defaults.ts b/packages/util/src/defaults.ts index 6acc5886b5c..185269dc3d4 100644 --- a/packages/util/src/defaults.ts +++ b/packages/util/src/defaults.ts @@ -57,33 +57,13 @@ const getDefaultsFromGlobal = (): FirebaseDefaults | undefined => * process.env.__FIREBASE_DEFAULTS_PATH__ */ const getDefaultsFromEnvVariable = (): FirebaseDefaults | undefined => { - if (typeof process === 'undefined') { + if (typeof process === 'undefined' || typeof process.env === 'undefined') { return; } const defaultsJsonString = process.env.__FIREBASE_DEFAULTS__; - const defaultsJsonPath = process.env.__FIREBASE_DEFAULTS_PATH__; if (defaultsJsonString) { - if (defaultsJsonPath) { - console.warn( - `Values were provided for both __FIREBASE_DEFAULTS__ ` + - `and __FIREBASE_DEFAULTS_PATH__. __FIREBASE_DEFAULTS_PATH__ ` + - `will be ignored.` - ); - } return JSON.parse(defaultsJsonString); } - if (defaultsJsonPath && typeof require !== 'undefined') { - try { - // eslint-disable-next-line @typescript-eslint/no-require-imports - const json = require(defaultsJsonPath); - return json; - } catch (e) { - console.warn( - `Unable to read defaults from file provided to ` + - `__FIREBASE_DEFAULTS_PATH__: ${defaultsJsonPath}` - ); - } - } }; const getDefaultsFromCookie = (): FirebaseDefaults | undefined => {