aboutsummaryrefslogtreecommitdiff
path: root/middleware/edns_test.go
diff options
context:
space:
mode:
authorGravatar Miek Gieben <miek@miek.nl> 2016-04-09 11:13:04 +0100
committerGravatar Miek Gieben <miek@miek.nl> 2016-04-09 11:13:04 +0100
commitdb3d689a8ac2bec199e5643394ffa779341acde0 (patch)
tree01e8a18adb3bb7c2d6bf73b9ab2964452136e97f /middleware/edns_test.go
parent16c035731c528f986a93d01b18244cad7d71e725 (diff)
downloadcoredns-db3d689a8ac2bec199e5643394ffa779341acde0.tar.gz
coredns-db3d689a8ac2bec199e5643394ffa779341acde0.tar.zst
coredns-db3d689a8ac2bec199e5643394ffa779341acde0.zip
EDNS: return error on wrong version. (#95)
Split up the previous changes a bit. This PR only returns the expected error when the received packet has the wrong EDNS version. EDNS0 handling in the middleware needs a nicer abstraction, like ReflectEdns() or something.
Diffstat (limited to 'middleware/edns_test.go')
-rw-r--r--middleware/edns_test.go37
1 files changed, 37 insertions, 0 deletions
diff --git a/middleware/edns_test.go b/middleware/edns_test.go
new file mode 100644
index 000000000..7b4e6fc66
--- /dev/null
+++ b/middleware/edns_test.go
@@ -0,0 +1,37 @@
+package middleware
+
+import (
+ "testing"
+
+ "github.com/miekg/dns"
+)
+
+func TestEdns0Version(t *testing.T) {
+ m := ednsMsg()
+ m.Extra[0].(*dns.OPT).SetVersion(2)
+
+ _, err := Edns0Version(m)
+ if err == nil {
+ t.Errorf("expected wrong version, but got OK")
+ }
+}
+
+func TestEdns0VersionNoEdns(t *testing.T) {
+ m := ednsMsg()
+ m.Extra = nil
+
+ _, err := Edns0Version(m)
+ if err != nil {
+ t.Errorf("expected no error, but got one: %s", err)
+ }
+}
+
+func ednsMsg() *dns.Msg {
+ m := new(dns.Msg)
+ m.SetQuestion("example.com.", dns.TypeA)
+ o := new(dns.OPT)
+ o.Hdr.Name = "."
+ o.Hdr.Rrtype = dns.TypeOPT
+ m.Extra = append(m.Extra, o)
+ return m
+}