Merge pull request #892 from superfly/trim-token-whitespace

trim whitespace when reading FLY_ACCESS_TOKEN
superfly · Apr 1, 2022 · 67a08e8 · 67a08e8
2 parents 130daa3 + 35d1ddb
commit 67a08e8
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/internal/config/config.go b/internal/config/config.go
@@ -1,6 +1,7 @@
 package config
 
 import (
+	"strings"
 	"sync"
 
 	"github.com/spf13/pflag"
@@ -85,6 +86,9 @@ func (cfg *Config) ApplyEnv() {
 	cfg.AccessToken = env.FirstOrDefault(cfg.AccessToken,
 		AccessTokenEnvKey, APITokenEnvKey)
 
+	// trim whitespace since it causes http errors when passsed to Docker auth
+	cfg.AccessToken = strings.TrimSpace(cfg.AccessToken)
+
 	cfg.VerboseOutput = env.IsTruthy(verboseOutputEnvKey) || cfg.VerboseOutput
 	cfg.JSONOutput = env.IsTruthy(jsonOutputEnvKey) || cfg.JSONOutput
 	cfg.LogGQLErrors = env.IsTruthy(logGQLEnvKey) || cfg.LogGQLErrors