Merge pull request #309 from equinix/env-flag-binding

fix: always bind flags to env vars when configuring a command
equinix · Jul 20, 2023 · 86ff3ab · 86ff3ab
2 parents 198710b + 4e1f20c
commit 86ff3ab
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/internal/cli/root.go b/internal/cli/root.go
@@ -137,9 +137,10 @@ func (c *Client) Config(cmd *cobra.Command) *viper.Viper {
 
  v.SetEnvPrefix(envPrefix)
  c.viper = v
- bindFlags(cmd, v)
  }
 
+ bindFlags(cmd, c.viper)
+
  flagToken := cmd.Flag("token").Value.String()
  envToken := cmd.Flag("auth-token").Value.String()
  // TODO: are we ok with this being configured by file too? yes?