Skip to content

Commit bf85695

Browse files
heschigopherbot
authored andcommitted
cmd/relui: actually fix publishing
I got the arguments to TrimPrefix backward, but more importantly, u.Query() returns a temporary copy. For golang/go#51797. Change-Id: I37a9df2adf47f1d72236d02761989d62e9991fc7 Reviewed-on: https://go-review.googlesource.com/c/build/+/411064 Reviewed-by: Cherry Mui <[email protected]> TryBot-Result: Gopher Robot <[email protected]> Run-TryBot: Heschi Kreinick <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]> Auto-Submit: Heschi Kreinick <[email protected]>
1 parent 09064db commit bf85695

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

cmd/relui/main.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -183,8 +183,10 @@ func publishFile(uploadURL string, auth buildlet.UserPass, f *relui.WebsiteFile)
183183
if err != nil {
184184
return fmt.Errorf("invalid website upload URL %q: %v", *websiteUploadURL, err)
185185
}
186-
u.Query().Set("user", strings.TrimPrefix("user-", auth.Username))
187-
u.Query().Set("key", auth.Password)
186+
q := u.Query()
187+
q.Set("user", strings.TrimPrefix(auth.Username, "user-"))
188+
q.Set("key", auth.Password)
189+
u.RawQuery = q.Encode()
188190
resp, err := http.Post(u.String(), "application/json", bytes.NewReader(req))
189191
if err != nil {
190192
return err

0 commit comments

Comments
 (0)