logging converted to *zap.SugaredLogger

Author: thejoeker12

httpclient/headers.go

@@ -4,12 +4,11 @@ package httpclient
 import (
 	"net/http"
 
-	"github.com/deploymenttheory/go-api-http-client/logger"
 	"go.uber.org/zap"
 )
 
 // CheckDeprecationHeader checks the response headers for the Deprecation header and logs a warning if present.
-func CheckDeprecationHeader(resp *http.Response, log logger.Logger) {
+func CheckDeprecationHeader(resp *http.Response, log *zap.SugaredLogger) {
 	deprecationHeader := resp.Header.Get("Deprecation")
 	if deprecationHeader != "" {
 		log.Warn("API endpoint is deprecated",

httpclient/request.go

@@ -171,7 +171,6 @@ func (c *Client) executeRequestWithRetries(method, endpoint string, body, out in
 			if apiErr := response.HandleAPIErrorResponse(resp, log); apiErr != nil {
 				err = apiErr
 			}
-			log.LogError("request_error", method, endpoint, resp.StatusCode, resp.Status, err, statusMessage)
 			break
 		}
 	}
@@ -292,7 +291,6 @@ func (c *Client) doRequest(ctx context.Context, method, endpoint string, body in
 	// }
 
 	// TODO review LogCookies
-	c.Logger.LogCookies("incoming", req, method, endpoint)
 
 	CheckDeprecationHeader(resp, c.Logger)