migrating to sugared logger

Author: thejoeker12

httpclient/client.go

@@ -18,12 +18,14 @@ import (
 	"github.com/deploymenttheory/go-api-http-client/redirecthandler"
 )
 
+const ()
+
 // TODO all struct comments
 
 // Master struct/object
 type Client struct {
 	// Config
-	config ClientConfig
+	config *ClientConfig
 
 	// Integration
 	Integration *APIIntegration
@@ -92,20 +94,24 @@ type ClientConfig struct {
 }
 
 // BuildClient creates a new HTTP client with the provided configuration.
-func (c ClientConfig) Build() (*Client, error) {
+func (c *ClientConfig) Build() (*Client, error) {
 	if c.Sugar == nil {
 		zapLogger, err := zap.NewProduction()
 		if err != nil {
 			return nil, err
 		}
 
 		c.Sugar = zapLogger.Sugar()
+		c.Sugar.Info("No logger provided. Defaulting to Sugared Zap Production Logger")
 	}
 
+	c.Sugar.Debug("validating configuration")
+
 	err := c.validateClientConfig()
 	if err != nil {
 		return nil, fmt.Errorf("invalid configuration: %v", err)
 	}
+	c.Sugar.Debug("configuration valid")
 
 	httpClient := &http.Client{
 		Timeout: c.CustomTimeout,
@@ -138,6 +144,7 @@ func (c ClientConfig) Build() (*Client, error) {
 	}
 
 	if len(client.config.CustomCookies) > 0 {
+		client.Sugar.Debug("setting custom cookies")
 		client.loadCustomCookies()
 	}
 

httpclient/cookies.go

@@ -18,6 +18,7 @@ func (c *Client) loadCustomCookies() error {
 	}
 
 	c.http.Jar.SetCookies(cookieUrl, c.config.CustomCookies)
+	c.Sugar.Debug("custom cookies set: %v", c.http.Jar.Cookies(cookieUrl))
 
 	return nil
 }

httpclient/headers.go

@@ -3,17 +3,12 @@ package httpclient
 
 import (
 	"net/http"
-
-	"go.uber.org/zap"
 )
 
 // CheckDeprecationHeader checks the response headers for the Deprecation header and logs a warning if present.
 func (c *Client) CheckDeprecationHeader(resp *http.Response) {
 	deprecationHeader := resp.Header.Get("Deprecation")
 	if deprecationHeader != "" {
-		c.Sugar.Warn("API endpoint is deprecated",
-			zap.String("Date", deprecationHeader),
-			zap.String("Endpoint", resp.Request.URL.String()),
-		)
+		c.Sugar.Warn("API endpoint is deprecated", deprecationHeader, resp.Request.URL.String())
 	}
 }