From 1f08853fc7d55dc5817eb08b3f001f2b0c8a5508 Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Sat, 16 Dec 2017 23:48:19 -0500 Subject: [PATCH] fix: handle case where DOKKU_DOCKERFILE_PORTS is an empty string --- .../triggers/network-compute-ports/network-compute-ports.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/plugins/network/src/triggers/network-compute-ports/network-compute-ports.go b/plugins/network/src/triggers/network-compute-ports/network-compute-ports.go index b2edbbbde..e83fc3620 100644 --- a/plugins/network/src/triggers/network-compute-ports/network-compute-ports.go +++ b/plugins/network/src/triggers/network-compute-ports/network-compute-ports.go @@ -5,6 +5,7 @@ import ( "fmt" "os" "strings" + "unicode/utf8" "github.com/dokku/dokku/plugins/common" "github.com/dokku/dokku/plugins/config" @@ -23,7 +24,10 @@ func main() { var dockerfilePorts []string if !isHerokuishContainer { - dockerfilePorts = strings.Split(config.GetWithDefault(appName, "DOKKU_DOCKERFILE_PORTS", ""), " ") + dokkuDockerfilePorts := strings.Trim(config.GetWithDefault(appName, "DOKKU_DOCKERFILE_PORTS", ""), " ") + if utf8.RuneCountInString(dokkuDockerfilePorts) > 0 { + dockerfilePorts = strings.Split(dokkuDockerfilePorts, " ") + } } var ports []string