log: render happydns.Identifier values via .String() in log messages

Author: nemunaire

internal/api/controller/checker.go

@@ -244,7 +244,7 @@ func (cc *CheckerController) TriggerCheck(c *gin.Context) {
 	} else {
 		go func() {
 			if _, err := cc.engine.RunExecution(context.WithoutCancel(c.Request.Context()), exec, plan, req.Options); err != nil {
-				log.Printf("async RunExecution error for checker %q execution %v: %v", cname, exec.Id, err)
+				log.Printf("async RunExecution error for checker %q execution %s: %v", cname, exec.Id.String(), err)
 			}
 		}()
 		c.JSON(http.StatusAccepted, exec)

internal/api/middleware/user.go

@@ -75,7 +75,7 @@ func SameUserHandler(c *gin.Context) {
 	user := c.MustGet("user").(*happydns.User)
 
 	if !bytes.Equal(user.Id, myuser.Id) {
-		log.Printf("%s: tries to do action as %s (logged %s)", c.ClientIP(), myuser.Id, user.Id)
+		log.Printf("%s: tries to do action as %s (logged %s)", c.ClientIP(), myuser.Id.String(), user.Id.String())
 		c.AbortWithStatusJSON(http.StatusForbidden, happydns.ErrorResponse{Message: "Not authorized"})
 		return
 	}

internal/usecase/checker/scheduler.go

@@ -627,7 +627,7 @@ func (s *Scheduler) NotifyDomainRemoved(domainID happydns.Identifier) {
 	s.mu.Unlock()
 
 	if n > 0 {
-		log.Printf("Scheduler: NotifyDomainRemoved(%s): removed %d jobs", domainID, n)
+		log.Printf("Scheduler: NotifyDomainRemoved(%s): removed %d jobs", domainID.String(), n)
 	}
 }
 
@@ -741,7 +741,7 @@ func (s *Scheduler) loadDomainServices(domain *happydns.Domain) []*happydns.Serv
 		}
 		zone, err := s.zoneStore.GetZone(domain.ZoneHistory[idx])
 		if err != nil {
-			log.Printf("Scheduler: failed to load zone %s for domain %s: %v", domain.ZoneHistory[idx], domain.DomainName, err)
+			log.Printf("Scheduler: failed to load zone %s for domain %s: %v", domain.ZoneHistory[idx].String(), domain.DomainName, err)
 			continue
 		}
 		for _, svcs := range zone.Services {

internal/usecase/orchestrator/zone_correction_applier.go

@@ -209,7 +209,7 @@ func (uc *ZoneCorrectionApplierUsecase) Apply(
 	if len(domain.ZoneHistory) > 1 {
 		prevZone, prevErr := uc.zoneGetter.Get(domain.ZoneHistory[1])
 		if prevErr != nil {
-			log.Printf("ReassociateMetadata: unable to load previous zone %s: %s (metadata will not be transferred)", domain.ZoneHistory[1], prevErr)
+			log.Printf("ReassociateMetadata: unable to load previous zone %s: %s (metadata will not be transferred)", domain.ZoneHistory[1].String(), prevErr)
 		} else {
 			zoneUC.ReassociateMetadata(prevZone.Services, services, domain.DomainName, defaultTTL)
 		}

internal/usecase/orchestrator/zone_importer.go

@@ -62,7 +62,7 @@ func (uc *ZoneImporterUsecase) Import(user *happydns.User, domain *happydns.Doma
 	if len(domain.ZoneHistory) > 0 {
 		prevZone, err := uc.zoneGetter.Get(domain.ZoneHistory[0])
 		if err != nil {
-			log.Printf("ReassociateMetadata: unable to load previous zone %s: %s (metadata will not be transferred)", domain.ZoneHistory[0], err)
+			log.Printf("ReassociateMetadata: unable to load previous zone %s: %s (metadata will not be transferred)", domain.ZoneHistory[0].String(), err)
 		} else {
 			zoneUC.ReassociateMetadata(prevZone.Services, services, domain.DomainName, defaultTTL)
 		}