3.0.3

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushrocks/qenv",
-  "version": "3.0.1",
+  "version": "3.0.3",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushrocks/qenv",
-  "version": "3.0.1",
+  "version": "3.0.3",
   "private": false,
   "description": "easy promised environments",
   "main": "dist/index.js",

ts/qenv.classes.qenv.ts

@@ -5,6 +5,10 @@ export interface IKeyValueObject {
   value: string;
 }
 
+/**
+ * class Qenv
+ * allows to make assertions about the environments while being more flexibel in how to meet them
+ */
 export class Qenv {
   public requiredEnvVars: string[] = [];
   public availableEnvVars: string[] = [];
@@ -25,8 +29,14 @@ export class Qenv {
     this.logger = loggerArg;
 
     // lets make sure paths are absolute
-    this.qenvFilePathAbsolute = plugins.path.join(plugins.path.resolve(qenvFileBasePathArg), 'qenv.yml');
-    this.envFilePathAbsolute = plugins.path.join(plugins.path.resolve(envFileBasePathArg), 'env.yml');
+    this.qenvFilePathAbsolute = plugins.path.join(
+      plugins.path.resolve(qenvFileBasePathArg),
+      'qenv.yml'
+    );
+    this.envFilePathAbsolute = plugins.path.join(
+      plugins.path.resolve(envFileBasePathArg),
+      'env.yml'
+    );
 
     this.getRequiredEnvVars();
     this.getAvailableEnvVars();
@@ -46,10 +56,18 @@ export class Qenv {
     }
   }
 
-  public getEnvVar(envVarName): string {
+  /**
+   * only gets an environment variable if it is required within a read qenv.yml file
+   * @param envVarName
+   */
+  public getEnvVarRequired(envVarName): string {
     return process.env[envVarName];
   }
 
+  /**
+   * tries to get any env var even if it is not required
+   * @param requiredEnvVar
+   */
   public getEnvVarOnDemand(requiredEnvVar: string): string {
     // lets determine the actual env yml
     let envYml;
@@ -62,6 +80,7 @@ export class Qenv {
     let envVar: string;
     let envFileVar: string;
     let dockerSecret: string;
+    let dockerSecretJson: string;
 
     // env var check
     if (process.env[requiredEnvVar]) {
@@ -74,6 +93,8 @@ export class Qenv {
       envFileVar = envYml[requiredEnvVar];
       this.availableEnvVars.push(requiredEnvVar);
     }
+
+    // docker secret check
     if (
       plugins.smartfile.fs.isDirectory('/run') &&
       plugins.smartfile.fs.isDirectory('/run/secrets') &&
@@ -82,10 +103,27 @@ export class Qenv {
       dockerSecret = plugins.smartfile.fs.toStringSync(`/run/secrets/${requiredEnvVar}`);
     }
 
-      if ((envVar && envFileVar) || (envVar && dockerSecret) || (envFileVar && dockerSecret)) {
+    // docker secret.json
+    if (
+      plugins.smartfile.fs.isDirectory('/run') &&
+      plugins.smartfile.fs.isDirectory('/run/secrets') &&
+      plugins.smartfile.fs.fileExists(`/run/secrets/secret.json`)
+    ) {
+      const secretObject = plugins.smartfile.fs.toObjectSync('/run/secrets/secret.json');
+      dockerSecret = secretObject[requiredEnvVar];
+    }
+
+    // warn if there is more than one candidate
+    let candidatesCounter = 0;
+    [envVar, envFileVar, dockerSecret, dockerSecretJson].forEach(candidate => {
+      if (candidate) {
+        candidatesCounter++;
+      }
+    });
+    if (candidatesCounter > 1) {
       this.logger.log(
         'warn',
-          `found multiple candidates for ${requiredEnvVar} Choosing in the order of envVar, envFileVar, dockerSecret`
+        `found multiple candidates for ${requiredEnvVar} Choosing in the order of envVar, envFileVar, dockerSecret, dockerSecretJson`
       );
     }
 
@@ -96,6 +134,8 @@ export class Qenv {
       chosenVar = envFileVar;
     } else if (dockerSecret) {
       chosenVar = dockerSecret;
+    } else if (dockerSecretJson) {
+      chosenVar = dockerSecretJson;
     }
     return chosenVar;
   }
@@ -104,20 +144,23 @@ export class Qenv {
    * gets the required env values
    */
   private getRequiredEnvVars = () => {
-    const qenvFile = plugins.smartfile.fs.toObjectSync(this.qenvFilePathAbsolute);
-    if (!qenvFile.required) {
-      this.logger.log('warn', `env File does not contain a 'required' Array!`);
+    let qenvFile: any = {};
+    if (plugins.smartfile.fs.fileExistsSync(this.qenvFilePathAbsolute)) {
+      qenvFile = plugins.smartfile.fs.toObjectSync(this.qenvFilePathAbsolute);
     }
+    if (!qenvFile || !qenvFile.required || !Array.isArray(qenvFile.required)) {
+      this.logger.log('warn', `env File does not contain a 'required' Array!`);
+    } else {
       for (const keyArg of Object.keys(qenvFile.required)) {
         this.requiredEnvVars.push(qenvFile.required[keyArg]);
       }
     }
+  };
 
   /**
    * gets the available env vars
    */
   private getAvailableEnvVars = () => {
-
     for (const requiredEnvVar of this.requiredEnvVars) {
       const chosenVar = this.getEnvVarOnDemand(requiredEnvVar);
       if (chosenVar) {
@@ -129,7 +172,7 @@ export class Qenv {
         });
       }
     }
-  }
+  };
 
   /**
    * gets missing env vars
@@ -142,5 +185,5 @@ export class Qenv {
       }
     }
     return missingEnvVars;
-  }
+  };
 }