fix: ignore empty string when setting endpoint

jooola · jooola · commit 70c0f36f4050 · 2025-03-12T09:08:12.000+01:00
diff --git a/hcloud/client.go b/hcloud/client.go
@@ -124,7 +124,9 @@ type ClientOption func(*Client)
 // WithEndpoint configures a Client to use the specified API endpoint.
 func WithEndpoint(endpoint string) ClientOption {
 	return func(client *Client) {
-		client.endpoint = strings.TrimRight(endpoint, "/")
+		if endpoint != "" {
+			client.endpoint = strings.TrimRight(endpoint, "/")
+		}
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,9 @@ type ClientOption func(*Client)`
`124`	`124`	`// WithEndpoint configures a Client to use the specified API endpoint.`
`125`	`125`	`func WithEndpoint(endpoint string) ClientOption {`
`126`	`126`	`return func(client *Client) {`
`127`		`- client.endpoint = strings.TrimRight(endpoint, "/")`
	`127`	`+ if endpoint != "" {`
	`128`	`+ client.endpoint = strings.TrimRight(endpoint, "/")`
	`129`	`+ }`
`128`	`130`	`}`
`129`	`131`	`}`
`130`	`132`