From 273db078b03f4fa4b8e3d71fb23719bba334aaa4 Mon Sep 17 00:00:00 2001 From: decentral1se Date: Sun, 5 Sep 2021 21:46:36 +0200 Subject: [PATCH] fix: bail out if app doesn't exist Closes https://git.coopcloud.tech/coop-cloud/go-abra/issues/67. Closes https://git.coopcloud.tech/coop-cloud/go-abra/pulls/69. Fix lifted from approach in https://git.coopcloud.tech/coop-cloud/go-abra/pulls/69. Thanks for @knoflook! --- cli/app/config.go | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/cli/app/config.go b/cli/app/config.go index 87c22137..1921fe6e 100644 --- a/cli/app/config.go +++ b/cli/app/config.go @@ -22,6 +22,15 @@ var appConfigCommand = &cli.Command{ internal.ShowSubcommandHelpAndError(c, errors.New("no app name provided")) } + appFiles, err := config.LoadAppFiles("") + if err != nil { + logrus.Fatal(err) + } + + if _, ok := appFiles[appName]; !ok { + logrus.Fatalf("'%s' doesn't exist?", appName) + } + ed, ok := os.LookupEnv("EDITOR") if !ok { edPrompt := &survey.Select{ @@ -33,11 +42,6 @@ var appConfigCommand = &cli.Command{ } } - appFiles, err := config.LoadAppFiles("") - if err != nil { - logrus.Fatal(err) - } - cmd := exec.Command(ed, appFiles[appName].Path) cmd.Stdin = os.Stdin cmd.Stdout = os.Stdout