Minimally addresses naggie#71: don't swallow errors.

xxxserxxx · Aug 10, 2022 · 8146b60 · 8146b60
1 parent 5adadaf
commit 8146b60
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/cmd/root.go b/cmd/root.go
@@ -39,8 +39,12 @@ var (
 		Run: func(cmd *cobra.Command, args []string) {
 			config := cli.MustLoadConfigFile()
 			server := cli.GetServer(config)
-			server.Up()
-			utils.ShellOut(config.PostUp, "PostUp")
+			if e := server.Up(); e != nil {
+				fmt.Printf("error bringing up the network: %s\n", e)
+			}
+			if e := utils.ShellOut(config.PostUp, "PostUp"); e != nil {
+				fmt.Printf("error bringing up the network: %s\n", e)
+			}
 		},
 	}
 
@@ -50,8 +54,12 @@ var (
 		Run: func(cmd *cobra.Command, args []string) {
 			config := cli.MustLoadConfigFile()
 			server := cli.GetServer(config)
-			server.DeleteLink()
-			utils.ShellOut(config.PostDown, "PostDown")
+			if e := server.DeleteLink(); e != nil {
+				fmt.Printf("error bringing up the network: %s\n", e)
+			}
+			if e := utils.ShellOut(config.PostDown, "PostDown"); e != nil {
+				fmt.Printf("error bringing up the network: %s\n", e)
+			}
 		},
 	}