Skip to content

Commit

Permalink
PR review fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
qdm12 committed Aug 21, 2024
1 parent 6c9198b commit 06092f8
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
4 changes: 2 additions & 2 deletions internal/configuration/settings/dot.go
Original file line number Diff line number Diff line change
Expand Up @@ -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))
Expand Down
2 changes: 1 addition & 1 deletion internal/configuration/settings/settings_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
1 change: 1 addition & 0 deletions internal/dns/loop.go
Original file line number Diff line number Diff line change
Expand Up @@ -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)

Expand Down

0 comments on commit 06092f8

Please sign in to comment.