Clean logging for modules

Signed-off-by: Mel <mel@rnrd.eu>

2 files changed, 11 insertions, 6 deletions

diff --git a/provider/dummy.go b/provider/dummy.go
index bec5907..1f64f6f 100644
--- a/provider/dummy.go
+++ b/provider/dummy.go
@@ -17,15 +17,12 @@ type dummy struct {
 var _ Provider = (*dummy)(nil)
 
 func newDummy(log *slog.Logger) (*dummy, error) {
-	return &dummy{log: log.With(slog.String("provider", "dummy"))}, nil
+	return &dummy{log: log}, nil
 }
 
-func (d *dummy) Ensure(zone, name string, rtype RecordType, addr netip.Addr) (string, bool, error) {
+func (d *dummy) Ensure(_ string, _ string, _ RecordType, addr netip.Addr) (string, bool, error) {
 	d.log.Info(
 		"asked to ensure record, but nothing happened",
-		slog.String("zone", zone),
-		slog.String("name", name),
-		slog.String("record", string(rtype)),
 		slog.String("addr", addr.String()),
 	)
 	return "", true, nil
diff --git a/provider/provider.go b/provider/provider.go
index ee31ba8..ce4a715 100644
--- a/provider/provider.go
+++ b/provider/provider.go
@@ -63,7 +63,15 @@ type Provider interface {
 	Ensure(zone, name string, rtype RecordType, addr netip.Addr) (string, bool, error)
 }
 
-func New(log *slog.Logger, cfg Config) (Provider, error) {
+func New(log *slog.Logger, name string, cfg Config) (Provider, error) {
+	log = log.With(
+		slog.Group(
+			"provider",
+			slog.String("name", name),
+			slog.String("kind", string(cfg.Kind)),
+		),
+	)
+
 	switch cfg.Kind {
 	case KindDummy:
 		return newDummy(log)