mirror of
				https://github.com/coredns/coredns.git
				synced 2025-10-31 02:03:20 -04:00 
			
		
		
		
	Fix the error handling. Log when we have an error during any of the transfer state. And if there isn't an error transfer the zones. Also fix the tests in test/ so we, at least, check the initial transfer. Update the docs to show more about how errors are handled. Ref #1400
		
			
				
	
	
		
			84 lines
		
	
	
		
			1.7 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			84 lines
		
	
	
		
			1.7 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| package test
 | |
| 
 | |
| import (
 | |
| 	"testing"
 | |
| 
 | |
| 	"github.com/coredns/coredns/plugin/proxy"
 | |
| 	"github.com/coredns/coredns/plugin/test"
 | |
| 	"github.com/coredns/coredns/request"
 | |
| 
 | |
| 	"github.com/miekg/dns"
 | |
| )
 | |
| 
 | |
| func TestEmptySecondaryZone(t *testing.T) {
 | |
| 	// Corefile that fails to transfer example.org.
 | |
| 	corefile := `example.org:0 {
 | |
| 		secondary {
 | |
| 			transfer from 127.0.0.1:1717
 | |
| 		}
 | |
| 	}
 | |
| `
 | |
| 
 | |
| 	i, udp, _, err := CoreDNSServerAndPorts(corefile)
 | |
| 	if err != nil {
 | |
| 		t.Fatalf("Could not get CoreDNS serving instance: %s", err)
 | |
| 	}
 | |
| 	defer i.Stop()
 | |
| 
 | |
| 	p := proxy.NewLookup([]string{udp})
 | |
| 	state := request.Request{W: &test.ResponseWriter{}, Req: new(dns.Msg)}
 | |
| 
 | |
| 	resp, err := p.Lookup(state, "www.example.org.", dns.TypeA)
 | |
| 	if err != nil {
 | |
| 		t.Fatal("Expected to receive reply, but didn't")
 | |
| 	}
 | |
| 	if resp.Rcode != dns.RcodeServerFailure {
 | |
| 		t.Fatalf("Expected reply to be a SERVFAIL, got %d", resp.Rcode)
 | |
| 	}
 | |
| }
 | |
| 
 | |
| func TestSecondaryZoneTransfer(t *testing.T) {
 | |
| 	name, rm, err := test.TempFile(".", exampleOrg)
 | |
| 	if err != nil {
 | |
| 		t.Fatalf("failed to create zone: %s", err)
 | |
| 	}
 | |
| 	defer rm()
 | |
| 
 | |
| 	corefile := `example.org:0 {
 | |
|        file ` + name + ` {
 | |
| 	       transfer to *
 | |
|        }
 | |
| }
 | |
| `
 | |
| 
 | |
| 	i, _, tcp, err := CoreDNSServerAndPorts(corefile)
 | |
| 	if err != nil {
 | |
| 		t.Fatalf("Could not get CoreDNS serving instance: %s", err)
 | |
| 	}
 | |
| 	defer i.Stop()
 | |
| 
 | |
| 	corefile = `example.org:0 {
 | |
| 		secondary {
 | |
| 			transfer from ` + tcp + `
 | |
| 		}
 | |
| }
 | |
| `
 | |
| 	i1, udp, _, err := CoreDNSServerAndPorts(corefile)
 | |
| 	if err != nil {
 | |
| 		t.Fatalf("Could not get CoreDNS serving instance: %s", err)
 | |
| 	}
 | |
| 	defer i1.Stop()
 | |
| 
 | |
| 	m := new(dns.Msg)
 | |
| 	m.SetQuestion("example.org.", dns.TypeSOA)
 | |
| 
 | |
| 	r, err := dns.Exchange(m, udp)
 | |
| 	if err != nil {
 | |
| 		t.Fatalf("Expected to receive reply, but didn't: %s", err)
 | |
| 	}
 | |
| 
 | |
| 	if len(r.Answer) == 0 {
 | |
| 		t.Fatalf("Expected answer section")
 | |
| 	}
 | |
| }
 |