TrueCloudLab/coredns
16
0
Fork 2
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

plugin/loop: add missing break (#1992)

Browse source 
The Onstartup loop wouldn't call break after the first success, thereby
sending the query multiple times and halting the process.

Some small other optimizations as well.

Signed-off-by: Miek Gieben <miek@miek.nl>
This commit is contained in:
Miek Gieben 2018-07-22 11:54:02 +01:00 • committed by GitHub
parent 170e1d6b66
commit b7ae8d4e86
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
plugin/loop/setup.go
View file

@ -37,10 +37,10 @@ func setup(c *caddy.Controller) error {
go func() { go func() {
deadline := time.Now().Add(30 * time.Second) deadline := time.Now().Add(30 * time.Second)
conf := dnsserver.GetConfig(c) conf := dnsserver.GetConfig(c)
lh := conf.ListenHosts[0]
addr := net.JoinHostPort(lh, conf.Port)
for time.Now().Before(deadline) { for time.Now().Before(deadline) {
lh := conf.ListenHosts[0]
addr := net.JoinHostPort(lh, conf.Port)
if _, err := l.exchange(addr); err != nil { if _, err := l.exchange(addr); err != nil {
time.Sleep(1 * time.Second) time.Sleep(1 * time.Second)
continue continue
@ -50,6 +50,8 @@ func setup(c *caddy.Controller) error {
time.Sleep(2 * time.Second) time.Sleep(2 * time.Second)
l.setDisabled() l.setDisabled()
}() }()
break
} }
l.setDisabled() l.setDisabled()
}() }()