From: Dmitri Shuralyov Date: Sun, 29 Mar 2020 03:25:18 +0000 (-0400) Subject: net/http: use DOMException.message property in error text X-Git-Tag: go1.15beta1~720 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=7bfac4c3ddde3dd906b344f141a9d09a5f855c77;p=gostls13.git net/http: use DOMException.message property in error text Previously, details about the underlying fetch error were not visible in the net/http error text: net/http: fetch() failed: When using the message property, they are: net/http: fetch() failed: Failed to fetch net/http: fetch() failed: The user aborted a request. Reference: https://developer.mozilla.org/en-US/docs/Web/API/DOMException/message. Change-Id: Iecf7c6bac01abb164731a4d5c9af6582c250a1a0 Reviewed-on: https://go-review.googlesource.com/c/go/+/226205 Reviewed-by: Johan Brandhorst --- diff --git a/src/net/http/roundtrip_js.go b/src/net/http/roundtrip_js.go index 4dd99651a7..e14f3f7152 100644 --- a/src/net/http/roundtrip_js.go +++ b/src/net/http/roundtrip_js.go @@ -157,7 +157,7 @@ func (t *Transport) RoundTrip(req *Request) (*Response, error) { }) defer success.Release() failure := js.FuncOf(func(this js.Value, args []js.Value) interface{} { - err := fmt.Errorf("net/http: fetch() failed: %s", args[0].String()) + err := fmt.Errorf("net/http: fetch() failed: %s", args[0].Get("message").String()) select { case errCh <- err: case <-req.Context().Done():