From d803dbb84a3fb757ebaa884ea00e8a72637eca91 Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Fri, 27 May 2022 23:44:46 -0400 Subject: [PATCH] fix: drop extra unnecessary validation of app name The app name is already validated when calling subcommands, so no need to validate it yet again here. --- plugins/config/environment.go | 8 -------- 1 file changed, 8 deletions(-) diff --git a/plugins/config/environment.go b/plugins/config/environment.go index 8e26f0a80..2f6ddf071 100644 --- a/plugins/config/environment.go +++ b/plugins/config/environment.go @@ -60,10 +60,6 @@ func newEnvFromString(rep string) (env *Env, err error) { //LoadAppEnv loads an environment for the given app func LoadAppEnv(appName string) (env *Env, err error) { - err = common.VerifyAppName(appName) - if err != nil { - return - } appfile, err := getAppFile(appName) if err != nil { return @@ -357,10 +353,6 @@ func loadFromFile(name string, filename string) (env *Env, err error) { } func getAppFile(appName string) (string, error) { - err := common.VerifyAppName(appName) - if err != nil { - return "", err - } return filepath.Join(common.MustGetEnv("DOKKU_ROOT"), appName, "ENV"), nil }