kotovalexarian-likes-github/moby--moby
1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00
Code Releases Activity

Merge pull request #2394 from arkodg/dns-lookup-case-insensitive

Browse source 
Make DNS records and queries case-insensitive
This commit is contained in:
Euan Harris 2019-06-20 16:37:07 +01:00 • committed by GitHub
commit bdd0b7bb40
3 changed files with 29 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

9
libnetwork/network.go
View file

@ -1381,14 +1381,18 @@ func delIPToName(ipMap setmatrix.SetMatrix, name, serviceID string, ip net.IP) {
} }
func addNameToIP(svcMap setmatrix.SetMatrix, name, serviceID string, epIP net.IP) { func addNameToIP(svcMap setmatrix.SetMatrix, name, serviceID string, epIP net.IP) {
svcMap.Insert(name, svcMapEntry{ // Since DNS name resolution is case-insensitive, Use the lower-case form
// of the name as the key into svcMap
lowerCaseName := strings.ToLower(name)
svcMap.Insert(lowerCaseName, svcMapEntry{
ip: epIP.String(), ip: epIP.String(),
serviceID: serviceID, serviceID: serviceID,
}) })
} }
func delNameToIP(svcMap setmatrix.SetMatrix, name, serviceID string, epIP net.IP) { func delNameToIP(svcMap setmatrix.SetMatrix, name, serviceID string, epIP net.IP) {
svcMap.Remove(name, svcMapEntry{ lowerCaseName := strings.ToLower(name)
svcMap.Remove(lowerCaseName, svcMapEntry{
ip: epIP.String(), ip: epIP.String(),
serviceID: serviceID, serviceID: serviceID,
}) })
@ -1956,6 +1960,7 @@ func (n *network) ResolveName(req string, ipType int) ([]net.IP, bool) {
} }
req = strings.TrimSuffix(req, ".") req = strings.TrimSuffix(req, ".")
req = strings.ToLower(req)
ipSet, ok := sr.svcMap.Get(req) ipSet, ok := sr.svcMap.Get(req)
if ipType == types.IPv6 { if ipType == types.IPv6 {

2
libnetwork/resolver.go
View file

@ -366,8 +366,8 @@ func (r *resolver) ServeDNS(w dns.ResponseWriter, query *dns.Msg) {
if query == nil || len(query.Question) == 0 { if query == nil || len(query.Question) == 0 {
return return
} }
name := query.Question[0].Name
name := query.Question[0].Name
switch query.Question[0].Qtype { switch query.Question[0].Qtype {
case dns.TypeA: case dns.TypeA:
resp, err = r.handleIPQuery(name, query, types.IPv4) resp, err = r.handleIPQuery(name, query, types.IPv4)

10
libnetwork/resolver_test.go
View file

@ -126,8 +126,11 @@ func TestDNSIPQuery(t *testing.T) {
r := NewResolver(resolverIPSandbox, false, sb.Key(), sb.(*sandbox)) r := NewResolver(resolverIPSandbox, false, sb.Key(), sb.(*sandbox))
// test name1's IP is resolved correctly with the default A type query // test name1's IP is resolved correctly with the default A type query
// Also make sure DNS lookups are case insensitive
names := []string{"name1", "NaMe1"}
for _, name := range names {
q := new(dns.Msg) q := new(dns.Msg)
q.SetQuestion("name1", dns.TypeA) q.SetQuestion(name, dns.TypeA)
r.(*resolver).ServeDNS(w, q) r.(*resolver).ServeDNS(w, q)
resp := w.GetResponse() resp := w.GetResponse()
checkNonNullResponse(t, resp) checkNonNullResponse(t, resp)
@ -143,12 +146,13 @@ func TestDNSIPQuery(t *testing.T) {
t.Fatal("Answer of type A not found") t.Fatal("Answer of type A not found")
} }
w.ClearResponse() w.ClearResponse()
}
// test MX query with name1 results in Success response with 0 answer records // test MX query with name1 results in Success response with 0 answer records
q = new(dns.Msg) q := new(dns.Msg)
q.SetQuestion("name1", dns.TypeMX) q.SetQuestion("name1", dns.TypeMX)
r.(*resolver).ServeDNS(w, q) r.(*resolver).ServeDNS(w, q)
resp = w.GetResponse() resp := w.GetResponse()
checkNonNullResponse(t, resp) checkNonNullResponse(t, resp)
t.Log("Response: ", resp.String()) t.Log("Response: ", resp.String())
checkDNSResponseCode(t, resp, dns.RcodeSuccess) checkDNSResponseCode(t, resp, dns.RcodeSuccess)