d2iq-archive / mesos-dns

DNS-based service discovery for Mesos.
https://mesosphere.github.com/mesos-dns
Apache License 2.0
483 stars 137 forks source link

Golang 1.11 LookupSRV can't resolve mesos-dns compressed responses. #530

Closed 2easy closed 5 years ago

2easy commented 5 years ago

Hi, with golang 1.11 I can't make it to resolve SRV records from mesos-dns. After some debugging I have discovered that golang does not allow for compressed SRV records (error message "compressed name in SRV resource data").


import (
    "log"
    "net"
)

func main() {
    _, _, err := net.LookupSRV("", "", "_my_service._udp.marathon.mesos.")
    log.Println(err)
}

Related issue: https://github.com/golang/go/issues/27546

jdef commented 5 years ago

To be clear, you're reporting a mesos-dns bug whereby mesos-dns is erroneously compressing part of the DNS response that it returns to the client?

jdef commented 5 years ago

Looks like we need to bump a dependency, this appears to have patched a while ago: https://github.com/miekg/dns/commit/00b8eaaab9411e1ff61e5030ef7e58bed412ad2a

2easy commented 5 years ago

Yup, that's it.