From 94f20cdd00149edf35e3963cc95d8a5cc8c7dd12 Mon Sep 17 00:00:00 2001 From: Nathan Williams Date: Tue, 17 Apr 2018 16:02:37 -0700 Subject: - reverse host field order (ip goes first) - fix host string split to permit IPv6 Signed-off-by: Nathan Williams Closes: #635 Approved by: rhatdan --- libpod/container_internal.go | 4 ++-- test/e2e/run_dns_test.go | 5 +++-- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/libpod/container_internal.go b/libpod/container_internal.go index e0411e028..414def120 100644 --- a/libpod/container_internal.go +++ b/libpod/container_internal.go @@ -901,8 +901,8 @@ func (c *Container) generateHosts() (string, error) { if len(c.config.HostAdd) > 0 { for _, host := range c.config.HostAdd { // the host format has already been verified at this point - fields := strings.Split(host, ":") - hosts += fmt.Sprintf("%s %s\n", fields[0], fields[1]) + fields := strings.SplitN(host, ":", 2) + hosts += fmt.Sprintf("%s %s\n", fields[1], fields[0]) } } return c.writeStringToRundir("hosts", hosts) diff --git a/test/e2e/run_dns_test.go b/test/e2e/run_dns_test.go index 27ca1e556..cfe6a34a2 100644 --- a/test/e2e/run_dns_test.go +++ b/test/e2e/run_dns_test.go @@ -62,10 +62,11 @@ var _ = Describe("Podman run dns", func() { }) It("podman run add host", func() { - session := podmanTest.Podman([]string{"run", "--add-host=foobar:1.1.1.1", ALPINE, "cat", "/etc/hosts"}) + session := podmanTest.Podman([]string{"run", "--add-host=foobar:1.1.1.1", "--add-host=foobaz:dead:beef:cafe", ALPINE, "cat", "/etc/hosts"}) session.WaitWithDefaultTimeout() Expect(session.ExitCode()).To(Equal(0)) - session.LineInOuputStartsWith("foobar 1.1.1.1") + session.LineInOuputStartsWith("1.1.1.1 foobar") + session.LineInOuputStartsWith("dead:beef:cafe foobaz") }) It("podman run add hostname", func() { -- cgit v1.2.3-54-g00ecf