v1.5.11

CHANGELOG.md

@@ -1,7 +1,7 @@
-## Version 1.5.9
+## Version 1.5.11
 
 ### Bugfix
-- Refactored clients to fix the problems with retries
+- Fix unhandled error in do() method for client and legacy-client
 
 
 

client.go

@@ -148,16 +148,17 @@ func (dc *DecortClient) do(req *http.Request) (*http.Response, error) {
 	// req = req.Clone(req.Context())
 	req.Body = io.NopCloser(bytes.NewBuffer(buf))
 	resp, err := dc.client.Do(req)
+	if err != nil || resp == nil {
+		return resp, err
+	}
 
-		// if err == nil {
 	if resp.StatusCode == 200 {
-				return resp, err
+		return resp, nil
 	}
+
 	respBytes, _ := io.ReadAll(resp.Body)
 	err = fmt.Errorf("%s", respBytes)
 	resp.Body.Close()
-		// }
-	// }
 	
 	return nil, fmt.Errorf("could not execute request: %w", err)
 }

legacy-client.go

@@ -145,16 +145,17 @@ func (ldc *LegacyDecortClient) do(req *http.Request) (*http.Response, error) {
 	// req = req.Clone(req.Context())
 	req.Body = io.NopCloser(bytes.NewBuffer(buf))
 	resp, err := ldc.client.Do(req)
+	if err != nil || resp == nil {
+		return resp, err
+	}
 
-		// if err == nil {
 	if resp.StatusCode == 200 {
-				return resp, err
+		return resp, nil
 	}
+
 	respBytes, _ := io.ReadAll(resp.Body)
 	err = fmt.Errorf("%s", respBytes)
 	resp.Body.Close()
-		// }
-	// }
 	
 	return nil, fmt.Errorf("could not execute request: %w", err)
 }