Merge pull request #285 from Cox-Automotive/print-log-format

Print log format

Author: americk0

changelog.txt

@@ -3,8 +3,7 @@
 
 Thanks for upgrading to the latest version of the ALKS CLI!
 
-* Removes node-fetch/request usage that resulted in punycode deprecation warnings.
-# Removes dependency on alks-node.
+* Improves logging around output formats
 
 Have feedback? https://github.com/Cox-Automotive/ALKS-CLI/issues
 

dist/changelog.txt

@@ -3,8 +3,7 @@
 
 Thanks for upgrading to the latest version of the ALKS CLI!
 
-* Removes node-fetch/request usage that resulted in punycode deprecation warnings.
-# Removes dependency on alks-node.
+* Improves logging around output formats
 
 Have feedback? https://github.com/Cox-Automotive/ALKS-CLI/issues
 

dist/src/lib/getKeyOutput.js

@@ -3,6 +3,7 @@ import moment from 'moment';
 import { Key } from '../model/keys';
 import { isWindows } from './isWindows';
 import { updateCreds } from './updateCreds';
+import { log } from './log';
 
 // if adding new output types be sure to update getOutputValues.ts
 export function getKeyOutput(
@@ -13,6 +14,8 @@ export function getKeyOutput(
 ) {
   const keyExpires = moment(key.expires).format();
 
+  log(`using output format: ${format}`);
+
   switch (format) {
     case 'docker': {
       return `-e AWS_ACCESS_KEY_ID=${key.accessKey} -e AWS_SECRET_ACCESS_KEY=${key.secretKey} -e AWS_SESSION_TOKEN=${key.sessionToken} -e AWS_SESSION_EXPIRES=${keyExpires}`;
@@ -73,6 +76,10 @@ export function getKeyOutput(
     case 'export': // fall through to default case
     case 'set':
     default: {
+      console.error(
+        'WARNING: Because this tool runs in a subshell, it cannot set environment variables in the parent shell. To use these keys, copy the commands printed below and run them in your current shell to have these environment variables set'
+      );
+
       const cmd = isWindows() || format === 'set' ? 'SET' : 'export';
 
       return `${cmd} AWS_ACCESS_KEY_ID=${key.accessKey} && ${cmd} AWS_SECRET_ACCESS_KEY=${key.secretKey} && ${cmd} AWS_SESSION_TOKEN=${key.sessionToken} && ${cmd} AWS_SESSION_EXPIRES=${keyExpires}`;