use whoami rather than $USER in secret_ops::docker_login

tomzo · tomzo · commit 739c416a4687 · 2019-05-01T15:42:09.000Z
This is more reliable
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,6 @@
+### 0.7.1 (2019-May-01)
+ * use `whoami` rather than `$USER` in `secret_ops::docker_login`
+
 ### 0.7.0 (2019-May-01)
 * added `secret_ops::docker_login`
 
diff --git a/src/secret-ops b/src/secret-ops
@@ -106,7 +106,7 @@ function secret_ops::insert_vault_token_gocd_yaml2 {
 }
 
 function secret_ops::docker_login {
-  vault_user_path=$USER
+  vault_user_path="$(whoami)"
   if [ $vault_user_path == "go" ]; then
     vault_user_path="gocd"
   fi

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ function secret_ops::insert_vault_token_gocd_yaml2 {`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`function secret_ops::docker_login {`
`109`		`- vault_user_path=$USER`
	`109`	`+ vault_user_path="$(whoami)"`
`110`	`110`	`if [ $vault_user_path == "go" ]; then`
`111`	`111`	`vault_user_path="gocd"`
`112`	`112`	`fi`