1
0
mirror of https://github.com/golang/go synced 2024-11-18 14:54:40 -07:00

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 <andybons@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
This commit is contained in:
Andrew Bonventre 2018-10-09 17:10:18 -04:00
parent afb03721b5
commit 9ae4309624

View File

@ -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))
}