From ab4b17b6cadff3e806bfb3d46da47a9e4f4d2a0c Mon Sep 17 00:00:00 2001 From: Quentin McGaw Date: Sat, 18 May 2024 11:39:40 +0000 Subject: [PATCH] PR review fixes --- internal/configuration/settings/dot.go | 4 ++-- internal/configuration/settings/settings_test.go | 2 +- internal/dns/loop.go | 1 + 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/internal/configuration/settings/dot.go b/internal/configuration/settings/dot.go index 49fa0dc76..157bfc3ce 100644 --- a/internal/configuration/settings/dot.go +++ b/internal/configuration/settings/dot.go @@ -127,9 +127,9 @@ func (d DoT) toLinesNode() (node *gotree.Node) { } node.Appendf("Update period: %s", update) - authServers := node.Appendf("Authoritative servers:") + upstreamResolvers := node.Appendf("Upstream resolvers:") for _, provider := range d.Providers { - authServers.Appendf(provider) + upstreamResolvers.Appendf(provider) } node.Appendf("Caching: %s", gosettings.BoolToYesNo(d.Caching)) diff --git a/internal/configuration/settings/settings_test.go b/internal/configuration/settings/settings_test.go index af8b31e13..d967f5b7c 100644 --- a/internal/configuration/settings/settings_test.go +++ b/internal/configuration/settings/settings_test.go @@ -43,7 +43,7 @@ func Test_Settings_String(t *testing.T) { | └── DNS over TLS settings: | ├── Enabled: yes | ├── Update period: every 24h0m0s -| ├── Authoritative servers: +| ├── Upstream resolvers: | | └── Cloudflare | ├── Caching: yes | ├── IPv6: no diff --git a/internal/dns/loop.go b/internal/dns/loop.go index 71e384f8d..f37766e28 100644 --- a/internal/dns/loop.go +++ b/internal/dns/loop.go @@ -43,6 +43,7 @@ func NewLoop(settings settings.DNS, stop := make(chan struct{}) stopped := make(chan struct{}) updateTicker := make(chan struct{}) + statusManager := loopstate.New(constants.Stopped, start, running, stop, stopped) state := state.New(statusManager, settings, updateTicker)