Mirrors/coredns
1
0
Fork 0
mirror of https://github.com/coredns/coredns.git synced 2025-10-27 16:24:19 -04:00
Code Issues Packages Projects Releases Wiki Activity

Revert "Validate zone during normalization (#3165)" (#3167)

Browse Source 
This reverts commit f888c5f7f6.
This commit is contained in:
Yong Tang
2019-08-21 08:01:45 -07:00
committed by Chris O'Haver
parent f888c5f7f6
commit 1b3fb834da
2 changed files with 1 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
core/dnsserver/address.go
View File

@@ -3,7 +3,6 @@ package dnsserver
import ( import (
"fmt" "fmt"
"net" "net"
"net/url"
"strings" "strings"
"github.com/coredns/coredns/plugin" "github.com/coredns/coredns/plugin"
@@ -53,13 +52,7 @@ func normalizeZone(str string) (zoneAddr, error) {
} }
} }
z := zoneAddr{Zone: dns.Fqdn(host), Port: port, Transport: trans, IPNet: ipnet} return zoneAddr{Zone: dns.Fqdn(host), Port: port, Transport: trans, IPNet: ipnet}, nil
_, err = url.ParseRequestURI(z.String())
if err != nil {
return zoneAddr{}, err
}
return z, nil
} }
// SplitProtocolHostPort splits a full formed address like "dns://[::1]:53" into parts. // SplitProtocolHostPort splits a full formed address like "dns://[::1]:53" into parts.

1
core/dnsserver/address_test.go
View File

@@ -28,7 +28,6 @@ func TestNormalizeZone(t *testing.T) {
{"https://.:8443", "https://.:8443", false}, {"https://.:8443", "https://.:8443", false},
{"https://..", "://:", true}, {"https://..", "://:", true},
{"https://.:", "://:", true}, {"https://.:", "://:", true},
{"dns://.:1053{.:53", "://:", true},
} { } {
addr, err := normalizeZone(test.input) addr, err := normalizeZone(test.input)
actual := addr.String() actual := addr.String()