From 7bfac4c3ddde3dd906b344f141a9d09a5f855c77 Mon Sep 17 00:00:00 2001 From: Dmitri Shuralyov Date: Sat, 28 Mar 2020 23:25:18 -0400 Subject: [PATCH] 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 --- src/net/http/roundtrip_js.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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(): -- 2.50.0