diff options
| author | Johan Brandhorst <johan.brandhorst@gmail.com> | 2018-08-24 12:10:01 +0100 |
|---|---|---|
| committer | Richard Musiol <neelance@gmail.com> | 2018-09-23 19:59:57 +0000 |
| commit | ce536837d8e53f1bf0c7ef450d4580d19f7d6f52 (patch) | |
| tree | 4a28b89f53171376c31842b7d186ad7d2d1a703d /src | |
| parent | fdefabadf0a2cb99accb2afe49eafce0eaeb53a7 (diff) | |
| download | go-ce536837d8e53f1bf0c7ef450d4580d19f7d6f52.tar.xz | |
net/http: ensure null body in Fetch response is not read
The Fetch API returns a null body if there is no response body,
on browsers that support streaming the response body. This
change ensures we check for both undefined and null bodies
before attempting to read the body.
Fixes #27196
Change-Id: I0da86b61284fe394418b4b431495e715a037f335
Reviewed-on: https://go-review.googlesource.com/131236
Reviewed-by: Richard Musiol <neelance@gmail.com>
Run-TryBot: Richard Musiol <neelance@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/net/http/roundtrip_js.go | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/net/http/roundtrip_js.go b/src/net/http/roundtrip_js.go index 16b7b891c8..38e4f5573e 100644 --- a/src/net/http/roundtrip_js.go +++ b/src/net/http/roundtrip_js.go @@ -116,7 +116,9 @@ func (t *Transport) RoundTrip(req *Request) (*Response, error) { b := result.Get("body") var body io.ReadCloser - if b != js.Undefined() { + // The body is undefined when the browser does not support streaming response bodies (Firefox), + // and null in certain error cases, i.e. when the request is blocked because of CORS settings. + if b != js.Undefined() && b != js.Null() { body = &streamReader{stream: b.Call("getReader")} } else { // Fall back to using ArrayBuffer |