diff --git a/util/dnsname/dnsname.go b/util/dnsname/dnsname.go index 56ee0baea..905e90f5f 100644 --- a/util/dnsname/dnsname.go +++ b/util/dnsname/dnsname.go @@ -24,13 +24,16 @@ func ToFQDN(s string) (FQDN, error) { if isValidFQDN(s) { return FQDN(s), nil } - if len(s) == 0 { + if len(s) == 0 || s == "." { return FQDN("."), nil } if s[len(s)-1] == '.' { s = s[:len(s)-1] } + if s[0] == '.' { + s = s[1:] + } if len(s) > maxNameLength { return "", fmt.Errorf("%q is too long to be a DNS name", s) } diff --git a/util/dnsname/dnsname_test.go b/util/dnsname/dnsname_test.go index 3df40dded..df3ea30b6 100644 --- a/util/dnsname/dnsname_test.go +++ b/util/dnsname/dnsname_test.go @@ -20,11 +20,12 @@ func TestFQDN(t *testing.T) { {".", ".", false, 0}, {"foo.com", "foo.com.", false, 2}, {"foo.com.", "foo.com.", false, 2}, + {".foo.com.", "foo.com.", false, 2}, + {".foo.com", "foo.com.", false, 2}, {"com", "com.", false, 1}, {"www.tailscale.com", "www.tailscale.com.", false, 3}, {"aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa.com", "", true, 0}, {strings.Repeat("aaaaa.", 60) + "com", "", true, 0}, - {".com", "", true, 0}, {"foo..com", "", true, 0}, }