minor logging fixes

Signed-off-by: Jonathan Holloway <[email protected]>

pkg/clients/imagebuilder/client.go

@@ -455,7 +455,6 @@ func (c *Client) GetComposeStatus(jobID string) (*ComposeStatus, error) {
 		c.log.WithFields(log.Fields{
 			"statusCode":   res.StatusCode,
 			"responseBody": string(body),
-			"error":        err,
 		}).Error("Error compose status HTTP response not StatusOK")
 
 		return nil, fmt.Errorf("request for status was not successful")
@@ -473,8 +472,7 @@ func (c *Client) GetComposeStatus(jobID string) (*ComposeStatus, error) {
 		c.log.WithFields(log.Fields{
 			"statusCode":   res.StatusCode,
 			"responseBody": string(body),
-			"error":        err,
-		}).Info("Image Builder ComposeStatus Response")
+		}).Info("ImageBuilder ComposeStatus successful response")
 	}
 
 	return cs, nil

pkg/services/images.go

@@ -12,6 +12,7 @@ import (
 	"fmt"
 	"io"
 	"net/http"
+	"net/url"
 	"os"
 	"os/signal"
 	"path/filepath"
@@ -891,7 +892,8 @@ func (s *ImageService) CreateRepoForImage(ctx context.Context, img *models.Image
 		return nil, err
 	}
 
-	s.log.WithField("url", repository.URL).Info("OSTree repo is ready")
+	parsedURL, _ := url.Parse(repository.URL)
+	s.log.WithField("url", parsedURL.Redacted()).Info("OSTree repo is ready")
 
 	return repository, nil
 }

pkg/services/repostore/pulpstore.go

@@ -197,7 +197,7 @@ func fileRepoImport(ctx context.Context, pulpService *pulp.PulpService, sourceUR
 		return "", "", err
 	}
 
-	log.WithContext(ctx).Info("File repo found or created: ", fileRepo)
+	log.WithContext(ctx).WithField("file_repo", fileRepo).Info("File repo found or created")
 
 	artifact, version, err := pulpService.FileRepositoriesImport(ctx, fileRepo, sourceURL)
 	if err != nil {