Release 4.3.1. Fixed secureConfig.fetchFromS3ByEnvVar not taking bu…

…cket name by env var.

dist/secureConfig/index.d.ts

@@ -1,4 +1,4 @@
 import "babel-polyfill";
 export { AuthenticationConfig } from "./AuthenticationConfig";
 export declare function fetchFromS3<T>(bucket: string, key: string): Promise<T>;
-export declare function fetchFromS3ByEnvVar<T>(bucket: string, envVar: string): Promise<T>;
+export declare function fetchFromS3ByEnvVar<T>(bucketEnvVar: string, keyEnvVar: string): Promise<T>;

package.json

@@ -1,6 +1,6 @@
 {
   "name": "giftbit-cassava-routes",
-  "version": "4.3.0",
+  "version": "4.3.1",
   "description": "Private Giftbit routes for use with Cassava.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

src/secureConfig/index.ts

@@ -26,12 +26,15 @@ export async function fetchFromS3<T>(bucket: string, key: string): Promise<T> {
     }
 }
 
-export async function fetchFromS3ByEnvVar<T>(bucket: string, envVar: string): Promise<T> {
-    if (!process || !process.env[envVar]) {
-        console.error(`${envVar} is not set.  The secure config item cannot be fetched.`);
+export async function fetchFromS3ByEnvVar<T>(bucketEnvVar: string, keyEnvVar: string): Promise<T> {
+    if (!process || !process.env[bucketEnvVar]) {
+        console.error(`${bucketEnvVar} is not set.  The secure config item cannot be fetched.`);
+        return null;
+    }
+    if (!process || !process.env[keyEnvVar]) {
+        console.error(`${keyEnvVar} is not set.  The secure config item cannot be fetched.`);
         return null;
     }
 
-    console.log(`Secure config env var ${envVar} = ${process.env[envVar]}.`);
-    return await fetchFromS3<T>(bucket, process.env[envVar]);
+    return await fetchFromS3<T>(process.env[bucketEnvVar], process.env[keyEnvVar]);
 }