github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-28 14:07:14 +00:00
Code Issues Projects Releases Wiki Activity

remove trailing dots from the parsed searches from host resolv.conf

Browse Source
This commit is contained in:
Fernando Crespo Gravalos 2019-09-24 16:30:22 +02:00 committed by Fernando Crespo Grávalos
parent 0541d0bb79
commit a215a88d91
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
pkg/kubelet/network/dns/dns.go
View File

@ -230,7 +230,11 @@ func parseResolvConf(reader io.Reader) (nameservers []string, searches []string,
} }
} }
if fields[0] == "search" { if fields[0] == "search" {
searches = fields[1:] trimTrailingDot := []string{}
for _, s := range fields[1:] {
trimTrailingDot = append(trimTrailingDot, strings.TrimSuffix(s, "."))
}
searches = trimTrailingDot
} }
if fields[0] == "options" { if fields[0] == "options" {
options = fields[1:] options = fields[1:]

1
pkg/kubelet/network/dns/dns_test.go
View File

@ -74,6 +74,7 @@ func TestParseResolvConf(t *testing.T) {
{"search ", []string{}, []string{}, []string{}, false}, // search empty {"search ", []string{}, []string{}, []string{}, false}, // search empty
{"search foo", []string{}, []string{"foo"}, []string{}, false}, {"search foo", []string{}, []string{"foo"}, []string{}, false},
{"search foo bar", []string{}, []string{"foo", "bar"}, []string{}, false}, {"search foo bar", []string{}, []string{"foo", "bar"}, []string{}, false},
{"search foo. bar", []string{}, []string{"foo", "bar"}, []string{}, false},
{"search foo bar bat\n", []string{}, []string{"foo", "bar", "bat"}, []string{}, false}, {"search foo bar bat\n", []string{}, []string{"foo", "bar", "bat"}, []string{}, false},
{"search foo\nsearch bar", []string{}, []string{"bar"}, []string{}, false}, {"search foo\nsearch bar", []string{}, []string{"bar"}, []string{}, false},
{"nameserver 1.2.3.4\nsearch foo bar", []string{"1.2.3.4"}, []string{"foo", "bar"}, []string{}, false}, {"nameserver 1.2.3.4\nsearch foo bar", []string{"1.2.3.4"}, []string{"foo", "bar"}, []string{}, false},