always return the response body

scriptnull · scriptnull · commit 65a97a3305fb · 2025-03-18T14:31:59.000+05:30
diff --git a/client.go b/client.go
@@ -55,15 +55,16 @@ func (c *Client) Do(ctx context.Context, q Queryable) (*bytes.Buffer, error) {
 	}
 	defer resp.Body.Close()
 
+	var respBytes bytes.Buffer
+	_, err = io.Copy(&respBytes, resp.Body)
+
 	switch {
 	case resp.StatusCode > 299 && resp.StatusCode < 399:
-		return nil, fmt.Errorf("%w: %d", ErrHTTPRedirect, resp.StatusCode)
+		err = fmt.Errorf("%w: %d", ErrHTTPRedirect, resp.StatusCode)
 	case resp.StatusCode > 399:
-		return nil, fmt.Errorf("%w: %d", ErrHTTPFailedStatus, resp.StatusCode)
+		err = fmt.Errorf("%w: %d", ErrHTTPFailedStatus, resp.StatusCode)
 	}
 
-	var respBytes bytes.Buffer
-	_, err = io.Copy(&respBytes, resp.Body)
 	return &respBytes, err
 }
 

Original file line number	Diff line number	Diff line change
`@@ -55,15 +55,16 @@ func (c Client) Do(ctx context.Context, q Queryable) (bytes.Buffer, error) {`
`55`	`55`	`}`
`56`	`56`	`defer resp.Body.Close()`
`57`	`57`
	`58`	`+ var respBytes bytes.Buffer`
	`59`	`+ _, err = io.Copy(&respBytes, resp.Body)`
	`60`	`+`
`58`	`61`	`switch {`
`59`	`62`	`case resp.StatusCode > 299 && resp.StatusCode < 399:`
`60`		`- return nil, fmt.Errorf("%w: %d", ErrHTTPRedirect, resp.StatusCode)`
	`63`	`+ err = fmt.Errorf("%w: %d", ErrHTTPRedirect, resp.StatusCode)`
`61`	`64`	`case resp.StatusCode > 399:`
`62`		`- return nil, fmt.Errorf("%w: %d", ErrHTTPFailedStatus, resp.StatusCode)`
	`65`	`+ err = fmt.Errorf("%w: %d", ErrHTTPFailedStatus, resp.StatusCode)`
`63`	`66`	`}`
`64`	`67`
`65`		`- var respBytes bytes.Buffer`
`66`		`- _, err = io.Copy(&respBytes, resp.Body)`
`67`	`68`	`return &respBytes, err`
`68`	`69`	`}`
`69`	`70`