Env contains distributable flag (elastic#40361)

* env contains distributable flag * make sure flag is always boolean
chrisronline · Jul 11, 2019 · e082e57 · e082e57
1 parent d429cfc
commit e082e57
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/src/core/server/config/__snapshots__/env.test.ts.snap b/src/core/server/config/__snapshots__/env.test.ts.snap
diff --git a/src/core/server/config/env.ts b/src/core/server/config/env.ts
@@ -28,6 +28,7 @@ export interface PackageInfo {
   branch: string;
   buildNum: number;
   buildSha: string;
+  dist: boolean;
 }
 
 export interface EnvironmentMode {
@@ -132,12 +133,13 @@ export class Env {
       prod: !isDevMode,
     });
 
-    const isKibanaDistributable = pkg.build && pkg.build.distributable === true;
+    const isKibanaDistributable = Boolean(pkg.build && pkg.build.distributable === true);
     this.packageInfo = Object.freeze({
       branch: pkg.branch,
       buildNum: isKibanaDistributable ? pkg.build.number : Number.MAX_SAFE_INTEGER,
       buildSha: isKibanaDistributable ? pkg.build.sha : 'XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX',
       version: pkg.version,
+      dist: isKibanaDistributable,
     });
   }
 }
diff --git a/src/core/server/plugins/discovery/plugin_manifest_parser.test.ts b/src/core/server/plugins/discovery/plugin_manifest_parser.test.ts
@@ -31,6 +31,7 @@ const packageInfo = {
   buildNum: 1,
   buildSha: '',
   version: '7.0.0-alpha1',
+  dist: false,
 };
 
 afterEach(() => {