revert JoinPath in createHTTPRequest because the input path could contain a query string

peterguy · peterguy · commit ed3146553e5e · 2026-03-09T18:51:33.000-07:00
diff --git a/internal/api/api.go b/internal/api/api.go
@@ -173,7 +173,8 @@ func (c *client) NewHTTPRequest(ctx context.Context, method, p string, body io.R
 }
 
 func (c *client) createHTTPRequest(ctx context.Context, method, p string, body io.Reader) (*http.Request, error) {
-	req, err := http.NewRequestWithContext(ctx, method, c.opts.EndpointURL.JoinPath(p).String(), body)
+	// Can't use c.opts.EndpointURL.JoinPaht(p) here because `p` could contain a query string
+	req, err := http.NewRequestWithContext(ctx, method, c.opts.EndpointURL.String()+"/"+p, body)
 	if err != nil {
 		return nil, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,8 @@ func (c *client) NewHTTPRequest(ctx context.Context, method, p string, body io.R`
`173`	`173`	`}`
`174`	`174`
`175`	`175`	`func (c client) createHTTPRequest(ctx context.Context, method, p string, body io.Reader) (http.Request, error) {`
`176`		`- req, err := http.NewRequestWithContext(ctx, method, c.opts.EndpointURL.JoinPath(p).String(), body)`
	`176`	+ // Can't use c.opts.EndpointURL.JoinPaht(p) here because `p` could contain a query string
	`177`	`+ req, err := http.NewRequestWithContext(ctx, method, c.opts.EndpointURL.String()+"/"+p, body)`
`177`	`178`	`if err != nil {`
`178`	`179`	`return nil, err`
`179`	`180`	`}`