Check for context error that is wrapped in url.Error

Evgeniy Makhrov · thaJeztah · commit 5a861b676cf5 · 2020-08-04T17:44:42.000+02:00
Signed-off-by: Evgeniy Makhrov &lt;e.makhrov@corp.badoo.com&gt;
(cherry picked from commit 8ccb46a521964ab80490318504d6e02ae766a912)
Signed-off-by: Sebastiaan van Stijn &lt;github@gone.nl&gt;
Upstream-commit: 89da709cb784b900145099cc638bbe8e26268ecc
Component: engine
diff --git a/components/engine/client/request.go b/components/engine/client/request.go
@@ -143,8 +143,7 @@ func (cli *Client) doRequest(ctx context.Context, req *http.Request) (serverResp
 
 		// Don't decorate context sentinel errors; users may be comparing to
 		// them directly.
-		switch err {
-		case context.Canceled, context.DeadlineExceeded:
+		if errors.Is(err, context.Canceled) || errors.Is(err, context.DeadlineExceeded) {
 			return serverResp, err
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -143,8 +143,7 @@ func (cli Client) doRequest(ctx context.Context, req http.Request) (serverResp`
`143`	`143`
`144`	`144`	`// Don't decorate context sentinel errors; users may be comparing to`
`145`	`145`	`// them directly.`
`146`		`- switch err {`
`147`		`- case context.Canceled, context.DeadlineExceeded:`
	`146`	`+ if errors.Is(err, context.Canceled) \|\| errors.Is(err, context.DeadlineExceeded) {`
`148`	`147`	`return serverResp, err`
`149`	`148`	`}`
`150`	`149`