From 6a4c0a16118d0b545c3c5f6a08ecad432940c1cf Mon Sep 17 00:00:00 2001 From: Mel Date: Thu, 20 Aug 2026 21:45:20 +0200 Subject: Clean logging for modules Signed-off-by: Mel --- provider/dummy.go | 7 ++----- provider/provider.go | 10 +++++++++- 2 files changed, 11 insertions(+), 6 deletions(-) (limited to 'provider') 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) -- cgit v1.3.1