mirror of
https://github.com/coredns/coredns.git
synced 2026-10-09 12:05:22 -04:00
* plugin/file: track zone mtime for reload_by_mtime Signed-off-by: git-jxj <65210887+git-jxj@users.noreply.github.com> * plugin/file: capture initial mtime before parsing Signed-off-by: git-jxj <65210887+git-jxj@users.noreply.github.com> * plugin/file: keep unloaded zones retryable after open error Signed-off-by: git-jxj <65210887+git-jxj@users.noreply.github.com> --------- Signed-off-by: git-jxj <65210887+git-jxj@users.noreply.github.com>
380 lines
9.6 KiB
Go
380 lines
9.6 KiB
Go
package file
|
|
|
|
import (
|
|
"bytes"
|
|
"context"
|
|
"fmt"
|
|
"io"
|
|
"os"
|
|
"path/filepath"
|
|
"strings"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/coredns/caddy"
|
|
"github.com/coredns/coredns/plugin/pkg/dnstest"
|
|
"github.com/coredns/coredns/plugin/pkg/fall"
|
|
"github.com/coredns/coredns/plugin/test"
|
|
|
|
"github.com/miekg/dns"
|
|
)
|
|
|
|
const dbRelative = `@ 500 IN SOA ns.example. hostmaster.example. 3 3600 600 86400 300
|
|
@ 500 IN NS ns.example.
|
|
foo 500 IN A 192.0.2.1
|
|
`
|
|
|
|
func TestFileParse(t *testing.T) {
|
|
zoneFileName1, rm, err := test.TempFile(".", dbMiekNL)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
defer rm()
|
|
|
|
zoneFileName2, rm, err := test.TempFile(".", dbDnssexNLSigned)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
defer rm()
|
|
|
|
zoneFileName3, rm, err := test.TempFile(".", dbRelative)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
defer rm()
|
|
|
|
tests := []struct {
|
|
inputFileRules string
|
|
shouldErr bool
|
|
expectedZones Zones
|
|
expectedFallthrough fall.F
|
|
}{
|
|
{
|
|
`file ` + zoneFileName1 + ` miek.nl.`,
|
|
false,
|
|
Zones{Names: []string{"miek.nl."}},
|
|
fall.Zero,
|
|
},
|
|
{
|
|
`file ` + zoneFileName2 + ` dnssex.nl.`,
|
|
false,
|
|
Zones{Names: []string{"dnssex.nl."}},
|
|
fall.Zero,
|
|
},
|
|
{
|
|
`file ` + zoneFileName3 + ` 10.0.0.0/8`,
|
|
false,
|
|
Zones{Names: []string{"10.in-addr.arpa."}},
|
|
fall.Zero,
|
|
},
|
|
{
|
|
`file ` + zoneFileName3 + ` example.org. {
|
|
fallthrough
|
|
}`,
|
|
false,
|
|
Zones{Names: []string{"example.org."}},
|
|
fall.Root,
|
|
},
|
|
{
|
|
`file ` + zoneFileName3 + ` example.org. {
|
|
fallthrough www.example.org
|
|
}`,
|
|
false,
|
|
Zones{Names: []string{"example.org."}},
|
|
fall.F{Zones: []string{"www.example.org."}},
|
|
},
|
|
// errors.
|
|
{
|
|
`file ` + zoneFileName1 + ` miek.nl {
|
|
transfer from 127.0.0.1
|
|
}`,
|
|
true,
|
|
Zones{},
|
|
fall.Zero,
|
|
},
|
|
{
|
|
`file`,
|
|
true,
|
|
Zones{},
|
|
fall.Zero,
|
|
},
|
|
{
|
|
`file ` + zoneFileName3 + ` example.net. {
|
|
no_reload
|
|
}`,
|
|
true,
|
|
Zones{},
|
|
fall.Zero,
|
|
},
|
|
{
|
|
`file ` + zoneFileName3 + ` example.net. {
|
|
no_rebloat
|
|
}`,
|
|
true,
|
|
Zones{},
|
|
fall.Zero,
|
|
},
|
|
}
|
|
|
|
for i, test := range tests {
|
|
c := caddy.NewTestController("dns", test.inputFileRules)
|
|
actualZones, actualFallthrough, err := fileParse(c)
|
|
|
|
if err == nil && test.shouldErr {
|
|
t.Fatalf("Test %d expected errors, but got no error", i)
|
|
} else if err != nil && !test.shouldErr {
|
|
t.Fatalf("Test %d expected no errors, but got '%v'", i, err)
|
|
} else {
|
|
if len(actualZones.Names) != len(test.expectedZones.Names) {
|
|
t.Fatalf("Test %d expected %v, got %v", i, test.expectedZones.Names, actualZones.Names)
|
|
}
|
|
for j, name := range test.expectedZones.Names {
|
|
if actualZones.Names[j] != name {
|
|
t.Fatalf("Test %d expected %v for %d th zone, got %v", i, name, j, actualZones.Names[j])
|
|
}
|
|
}
|
|
if !actualFallthrough.Equal(test.expectedFallthrough) {
|
|
t.Errorf("Test %d expected fallthrough of %v, got %v", i, test.expectedFallthrough, actualFallthrough)
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestParseReload(t *testing.T) {
|
|
name, rm, err := test.TempFile(".", dbRelative)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
defer rm()
|
|
|
|
tests := []struct {
|
|
input string
|
|
reload time.Duration
|
|
}{
|
|
{
|
|
`file ` + name + ` example.org.`,
|
|
1 * time.Minute,
|
|
},
|
|
{
|
|
`file ` + name + ` example.org. {
|
|
reload 5s
|
|
}`,
|
|
5 * time.Second,
|
|
},
|
|
}
|
|
|
|
for i, test := range tests {
|
|
c := caddy.NewTestController("dns", test.input)
|
|
z, _, err := fileParse(c)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if x := z.Z["example.org."].ReloadInterval; x != test.reload {
|
|
t.Errorf("Test %d expected reload to be %s, but got %s", i, test.reload, x)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestFileParseSOAOrigin(t *testing.T) {
|
|
for _, owner := range []string{"test", "@", "test."} {
|
|
for _, explicit := range []bool{false, true} {
|
|
t.Run(fmt.Sprintf("%s/explicit=%t", owner, explicit), func(t *testing.T) {
|
|
contents := fmt.Sprintf(`%s 500 IN SOA ns1.outside.com. root.test 3 604800 86400 2419200 604800
|
|
%s 500 IN NS ns1.outside.com.
|
|
foo 500 IN A 1.1.1.1
|
|
`, owner, owner)
|
|
fileName := filepath.Join(t.TempDir(), "db.test")
|
|
if err := os.WriteFile(fileName, []byte(contents), 0644); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
corefile := fmt.Sprintf("file %q", filepath.ToSlash(fileName))
|
|
if explicit {
|
|
corefile += " test"
|
|
}
|
|
c := caddy.NewTestController("dns", corefile)
|
|
c.ServerBlockKeys = []string{"test:53"}
|
|
zones, _, err := fileParse(c)
|
|
if owner == "test" {
|
|
if err == nil || !strings.Contains(err.Error(), "SOA owner test.test. that does not match origin test.") {
|
|
t.Fatalf("expected mismatched SOA error, got %v", err)
|
|
}
|
|
return
|
|
}
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
f := File{Zones: zones}
|
|
for _, tc := range []test.Case{
|
|
{
|
|
Qname: "foo.test.", Qtype: dns.TypeA, Authoritative: true,
|
|
Answer: []dns.RR{test.A("foo.test. 500 IN A 1.1.1.1")},
|
|
Ns: []dns.RR{test.NS("test. 500 IN NS ns1.outside.com.")},
|
|
},
|
|
{
|
|
Qname: "bar.test.", Qtype: dns.TypeA, Rcode: dns.RcodeNameError, Authoritative: true,
|
|
Ns: []dns.RR{test.SOA("test. 500 IN SOA ns1.outside.com. root.test.test. 3 604800 86400 2419200 604800")},
|
|
},
|
|
} {
|
|
rec := dnstest.NewRecorder(&test.ResponseWriter{})
|
|
if _, err := f.ServeDNS(context.Background(), rec, tc.Msg()); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if err := test.SortAndCheck(rec.Msg, tc); err != nil {
|
|
t.Error(err)
|
|
}
|
|
}
|
|
})
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestFileParseRelativeZones(t *testing.T) {
|
|
fileName, rm, err := test.TempFile(".", dbRelative)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
defer rm()
|
|
c := caddy.NewTestController("dns", "file "+fileName+" example.org example.net")
|
|
zones, _, err := fileParse(c)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
for _, origin := range []string{"example.org.", "example.net."} {
|
|
z := zones.Z[origin]
|
|
if z == nil || z.SOA == nil || z.SOA.Hdr.Name != origin {
|
|
t.Fatalf("missing SOA at %s", origin)
|
|
}
|
|
if _, ok := z.Search("foo." + origin); !ok {
|
|
t.Errorf("missing relative A record in %s", origin)
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestFileParseReloadByMtimeInitializesMtime(t *testing.T) {
|
|
name, rm, err := test.TempFile(".", dbMiekNL)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
defer rm()
|
|
|
|
c := caddy.NewTestController("dns", "file "+name+" miek.nl. {\n\treload_by_mtime\n}")
|
|
zones, _, err := fileParse(c)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
fi, err := os.Stat(name)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
z := zones.Z["miek.nl."]
|
|
if z.file_mtime.IsZero() {
|
|
t.Fatal("file mtime was not initialized for reload_by_mtime")
|
|
}
|
|
if !z.file_mtime.Equal(fi.ModTime()) {
|
|
t.Fatalf("file mtime = %s, want %s", z.file_mtime, fi.ModTime())
|
|
}
|
|
}
|
|
|
|
func TestFileParseReloadByMtimeMissingFileLeavesBaselineUnset(t *testing.T) {
|
|
name := filepath.Join(t.TempDir(), "missing.db")
|
|
c := caddy.NewTestController("dns", "file "+name+" example.org. {\n\treload_by_mtime\n}")
|
|
zones, _, err := fileParse(c)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if !zones.Z["example.org."].file_mtime.IsZero() {
|
|
t.Fatal("missing initial file must leave the mtime baseline unset")
|
|
}
|
|
}
|
|
|
|
func TestFileParseReloadByMtimeRecoversLaterZoneAfterMissingFile(t *testing.T) {
|
|
dir := t.TempDir()
|
|
missing := filepath.Join(dir, "missing.db")
|
|
existing := filepath.Join(dir, "existing.db")
|
|
if err := os.WriteFile(existing, []byte(dbMiekNL), 0600); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
corefile := fmt.Sprintf(`file %s example.org. {
|
|
reload 10ms
|
|
reload_by_mtime
|
|
}
|
|
file %s miek.nl. {
|
|
reload 10ms
|
|
reload_by_mtime
|
|
}`, missing, existing)
|
|
zones, _, err := fileParse(caddy.NewTestController("dns", corefile))
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
z := zones.Z["miek.nl."]
|
|
if z == nil {
|
|
t.Fatal("later zone was not configured")
|
|
}
|
|
if err := z.Reload(nil); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
defer z.OnShutdown()
|
|
|
|
const wantSerial = 1282630057
|
|
for start := time.Now(); time.Since(start) < 2*time.Second; {
|
|
if z.SOASerialIfDefined() == wantSerial {
|
|
return
|
|
}
|
|
time.Sleep(2 * time.Millisecond)
|
|
}
|
|
t.Fatalf("later zone never loaded: SOA serial = %d, want %d", z.SOASerialIfDefined(), wantSerial)
|
|
}
|
|
|
|
func TestFileParseReloadByMtimeUsesOpenedFileMtime(t *testing.T) {
|
|
name, rm, err := test.TempFile(".", dbRelative)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
defer rm()
|
|
|
|
initialMtime := time.Now().Add(-2 * time.Hour).Truncate(time.Second)
|
|
if err := os.Chtimes(name, initialMtime, initialMtime); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
c := caddy.NewTestController("dns", "file "+name+" example.org. {\n\treload_by_mtime\n}")
|
|
zones, _, err := fileParseWithParser(c, func(reader io.Reader, origin, fileName string, serial int64) (*Zone, error) {
|
|
// Simulate the opened reader consuming the old zone before a writer
|
|
// replaces its contents while the initial load is still in progress.
|
|
oldContents, err := io.ReadAll(reader)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
newContents := strings.Replace(dbRelative, "192.0.2.1", "192.0.2.2", 1)
|
|
if err := os.WriteFile(fileName, []byte(newContents), 0644); err != nil {
|
|
return nil, err
|
|
}
|
|
newMtime := initialMtime.Add(time.Hour)
|
|
if err := os.Chtimes(fileName, newMtime, newMtime); err != nil {
|
|
return nil, err
|
|
}
|
|
return Parse(bytes.NewReader(oldContents), origin, fileName, serial)
|
|
})
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
z := zones.Z["example.org."]
|
|
entry, ok := z.Search("foo.example.org.")
|
|
if !ok || len(entry.Type(dns.TypeA)) != 1 || entry.Type(dns.TypeA)[0].(*dns.A).A.String() != "192.0.2.1" {
|
|
t.Fatal("initial load did not retain the old zone record")
|
|
}
|
|
if !z.file_mtime.Equal(initialMtime) {
|
|
t.Fatalf("initial mtime = %s, want opened file's %s", z.file_mtime, initialMtime)
|
|
}
|
|
fi, err := os.Stat(name)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if !fi.ModTime().After(z.file_mtime) {
|
|
t.Fatalf("changed zone mtime %s must be newer than loaded baseline %s", fi.ModTime(), z.file_mtime)
|
|
}
|
|
}
|