From 9ae43096244f067598960f85a9c4c8a3c4e30911 Mon Sep 17 00:00:00 2001 From: Andrew Bonventre Date: Tue, 9 Oct 2018 17:10:18 -0400 Subject: [PATCH] playground: set content-type header on non-appengine builds Currently, when a POST request is made using code with build tag +build !appengine it doesn't set the content-type header passed to the function. This was breaking the case where a url-encoded body (a code snippet's body and protocol version) was being POSTed to golang.org/compile since it never set the content-type of the request and the corresponding form values parsed out were empty as a result. Update golang/go#28080 Change-Id: I677ca01b2f5aecedbd13d1faa7a838ddc0199244 Reviewed-on: https://go-review.googlesource.com/c/140838 Run-TryBot: Andrew Bonventre Reviewed-by: Dmitri Shuralyov --- playground/local.go | 1 + 1 file changed, 1 insertion(+) diff --git a/playground/local.go b/playground/local.go index 452054bd4..741ab6f9e 100644 --- a/playground/local.go +++ b/playground/local.go @@ -19,6 +19,7 @@ func post(ctx context.Context, url, contentType string, body io.Reader) (*http.R if err != nil { return nil, fmt.Errorf("http.NewRequest: %v", err) } + req.Header.Set("Content-Type", contentType) return http.DefaultClient.Do(req.WithContext(ctx)) }