fix(core): Always attempt to trim CommandAPIPath of leading or trailing /

spbsoluble · spbsoluble · commit 45023c94e9be · 2025-01-17T07:39:05.000-08:00
diff --git a/auth_providers/auth_core.go b/auth_providers/auth_core.go
@@ -270,11 +270,12 @@ func (c *CommandAuthConfig) ValidateAuthConfig() error {
 	}
 	if c.CommandAPIPath == "" {
 		if apiPath, ok := os.LookupEnv(EnvKeyfactorAPIPath); ok {
-			c.CommandAPIPath = strings.Trim(apiPath, "/")
+			c.CommandAPIPath = apiPath
 		} else {
 			c.CommandAPIPath = DefaultCommandAPIPath
 		}
 	}
+	c.CommandAPIPath = strings.Trim(c.CommandAPIPath, "/")
 	if c.HttpClientTimeout <= 0 {
 		if timeout, ok := os.LookupEnv(EnvKeyfactorClientTimeout); ok {
 			configTimeout, tErr := strconv.Atoi(timeout)

Original file line number	Diff line number	Diff line change
`@@ -270,11 +270,12 @@ func (c *CommandAuthConfig) ValidateAuthConfig() error {`
`270`	`270`	`}`
`271`	`271`	`if c.CommandAPIPath == "" {`
`272`	`272`	`if apiPath, ok := os.LookupEnv(EnvKeyfactorAPIPath); ok {`
`273`		`- c.CommandAPIPath = strings.Trim(apiPath, "/")`
	`273`	`+ c.CommandAPIPath = apiPath`
`274`	`274`	`} else {`
`275`	`275`	`c.CommandAPIPath = DefaultCommandAPIPath`
`276`	`276`	`}`
`277`	`277`	`}`
	`278`	`+ c.CommandAPIPath = strings.Trim(c.CommandAPIPath, "/")`
`278`	`279`	`if c.HttpClientTimeout <= 0 {`
`279`	`280`	`if timeout, ok := os.LookupEnv(EnvKeyfactorClientTimeout); ok {`
`280`	`281`	`configTimeout, tErr := strconv.Atoi(timeout)`