Happy path of the left.

dtrejod · dtrejod · commit dcba04fd14bb · 2022-08-15T20:00:08.000-04:00
diff --git a/conjure-go-client/httpclient/client.go b/conjure-go-client/httpclient/client.go
@@ -102,10 +102,11 @@ func (c *clientImpl) Do(ctx context.Context, params ...RequestParam) (*http.Resp
 			break
 		}
 		resp, err = c.doOnce(ctx, retryURL, params...)
-		if err == nil {
-			break
+		if err != nil {
+			svc1log.FromContext(ctx).Debug("Retrying request", svc1log.Stacktrace(err))
+			continue
 		}
-		svc1log.FromContext(ctx).Debug("Retrying request", svc1log.Stacktrace(err))
+		break
 	}
 	if err != nil {
 		return nil, err

Original file line number	Diff line number	Diff line change
`@@ -102,10 +102,11 @@ func (c clientImpl) Do(ctx context.Context, params ...RequestParam) (http.Resp`
`102`	`102`	`break`
`103`	`103`	`}`
`104`	`104`	`resp, err = c.doOnce(ctx, retryURL, params...)`
`105`		`- if err == nil {`
`106`		`- break`
	`105`	`+ if err != nil {`
	`106`	`+ svc1log.FromContext(ctx).Debug("Retrying request", svc1log.Stacktrace(err))`
	`107`	`+ continue`
`107`	`108`	`}`
`108`		`- svc1log.FromContext(ctx).Debug("Retrying request", svc1log.Stacktrace(err))`
	`109`	`+ break`
`109`	`110`	`}`
`110`	`111`	`if err != nil {`
`111`	`112`	`return nil, err`