bitwarden/browser
1
0
Fork 0
mirror of https://github.com/bitwarden/browser synced 2025-12-06 00:13:28 +00:00
Code Issues Releases Wiki Activity

Ps/fix dev flags not defined (#6683)

Browse Source 
* Handle devFlags not defined

* Default dev flags to false
This commit is contained in:
Matt Gibson
2023-10-24 11:48:01 -04:00
committed by GitHub
parent 950e327477
commit c798c92c84
3 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
libs/common/src/platform/misc/flags.spec.ts
View File

@@ -36,8 +36,13 @@ describe("devFlagEnabled", () => {
process.env.ENV = "development";
});
it("returns true by default", () => {
expect(devFlagEnabled<any>("nonExistentFlag")).toBe(true);
it("returns false by default", () => {
expect(devFlagEnabled<any>("nonExistentFlag")).toBe(false);
});
it("returns false if devFlags is not defined", () => {
delete process.env.DEV_FLAGS;
expect(devFlagEnabled<any>("nonExistentFlag")).toBe(false);
});
it("returns true if enabled", () => {

2
libs/common/src/platform/misc/flags.ts
View File

@@ -45,7 +45,7 @@ export function devFlagEnabled<DevFlags extends SharedDevFlags>(flag: keyof DevF
}
const devFlags = getFlags<DevFlags>(process.env.DEV_FLAGS);
return devFlags[flag] == null || !!devFlags[flag];
return devFlags?.[flag] == null ? false : !!devFlags[flag];
}
/**