fix: Improved error logging and updated Cobra to v1.0.0

Author: karl-cardenas-coding

cmd/categories.go

@@ -7,6 +7,7 @@ import (
 
 	"github.com/jedib0t/go-pretty/table"
 	"github.com/karl-cardenas-coding/disaster-cli/library"
+	log "github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
 )
 
@@ -59,7 +60,14 @@ The returned category may be used with the --filter flag for the event cmd.`,
 		if outputFlag == "json" {
 			json, err := json.Marshal(records)
 			if err != nil {
-				fmt.Println(err)
+				log.WithFields(log.Fields{
+					"package":         "cmd",
+					"file":            "categories.go",
+					"parent_function": "Run",
+					"function":        "json.Marshal",
+					"error":           err,
+					"data":            fmt.Sprint(cmd, "./documentation/"),
+				}).Error("Error marshaling JSON", ISSUE_MSG)
 			}
 			os.Stdout.Write(json)
 		}

cmd/disaster.go

@@ -1,10 +1,10 @@
 package cmd
 
 import (
-	"fmt"
-	"log"
 	"os"
 
+	log "github.com/sirupsen/logrus"
+
 	"github.com/spf13/cobra"
 	"github.com/spf13/cobra/doc"
 )
@@ -19,6 +19,10 @@ var (
 	DownloadTempPath string
 )
 
+const (
+	ISSUE_MSG = " Please open up a Github issue to report this error! https://github.com/karl-cardenas-coding/disaster-cli"
+)
+
 var rootCmd = &cobra.Command{
 	Use:   "disaster",
 	Short: "A CLI tool for determining natural catastrophe near you, or a location specified",
@@ -28,13 +32,27 @@ var rootCmd = &cobra.Command{
 		if generateDocFlag {
 			err := doc.GenMarkdownTree(cmd, "./documentation/")
 			if err != nil {
-				log.Fatal(err)
+				log.WithFields(log.Fields{
+					"package":  "cmd",
+					"file":     "disaster.go",
+					"parent_function": "generateDocFlag",
+					"function": "doc.GenMarkdownTree"
+					"error":    err,
+					"data":     fmt.Sprint(cmd, "./documentation/"),
+				}).Fatal("Error generating markdown content", ISSUE_MSG)
 			}
 		}
 
 		err := cmd.Help()
 		if err != nil {
-			log.Fatal(err)
+			log.WithFields(log.Fields{
+				"package":  "cmd",
+				"file":     "disaster.go",
+				"parent_function": "generateDocFlag",
+				"function": "cmd.Help",
+				"error":    err,
+				"data":     nil,
+			}).Fatal("Error outputting help!", ISSUE_MSG)
 		}
 	},
 }
@@ -46,11 +64,26 @@ func init() {
 	eventsCmd.Flags().BoolVarP(&DisplayMapFlag, "display-map", "d", false, "Displays the Google Maps URL")
 	eventsCmd.Flags().StringSliceVarP(&FiltersFlag, "filter", "f", []string{}, "filter events by passing in categories (comma seperated")
 	updateCmd.Flags().StringVarP(&DownloadTempPath, "temp-location", "l", "", "Specify the temporary directory to use for the update process")
+
+	// Establish logging default
+	log.SetFormatter(&log.TextFormatter{
+		DisableColors: false,
+		FullTimestamp: true,
+	})
+	log.SetOutput(os.Stdout)
+
+	log.SetLevel(log.WarnLevel)
 }
 
 func Execute() {
 	if err := rootCmd.Execute(); err != nil {
-		fmt.Println(err)
+		log.WithFields(log.Fields{
+			"package":  "cmd",
+			"file":     "disaster.go",
+			"function": "Execute",
+			"error":    err,
+			"data":     nil,
+		}).Fatal("Error executing the CLI!", ISSUE_MSG)
 		os.Exit(1)
 	}
 }

cmd/events.go

@@ -7,6 +7,7 @@ import (
 
 	"github.com/jedib0t/go-pretty/table"
 	"github.com/karl-cardenas-coding/disaster-cli/library"
+	log "github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
 )
 
@@ -75,7 +76,14 @@ func outputJSON(records library.EventResponse, filters []string) {
 
 		json, err := json.Marshal(&list)
 		if err != nil {
-			fmt.Println(err)
+			log.WithFields(log.Fields{
+				"package":         "cmd",
+				"file":            "events.go",
+				"parent_function": "outputJSON",
+				"function":        "json.Marshal",
+				"error":           err,
+				"data":            list,
+			}).Error("Error marshalling JSON", ISSUE_MSG)
 		}
 		os.Stdout.Write(json)
 
@@ -85,7 +93,14 @@ func outputJSON(records library.EventResponse, filters []string) {
 
 		json, err := json.Marshal(&records)
 		if err != nil {
-			fmt.Println(err)
+			log.WithFields(log.Fields{
+				"package":         "cmd",
+				"file":            "events.go",
+				"parent_function": "outputJSON",
+				"function":        "json.Marshal",
+				"error":           err,
+				"data":            records,
+			}).Error("Error marshalling JSON", ISSUE_MSG)
 		}
 		os.Stdout.Write(json)