chore: remove panics

kldzj · May 7, 2023 · 4c0b3cf · 4c0b3cf
1 parent 7459d4d
commit 4c0b3cf
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/steam/util.go b/steam/util.go
@@ -4,6 +4,9 @@ import (
 	"net/http"
 	"net/url"
 	"time"
+
+	"github.com/kldzj/pzmod/util"
+	"github.com/spf13/cobra"
 )
 
 const (
@@ -19,7 +22,7 @@ func SetApiKey(key string) {
 
 func getApiKey() string {
 	if steamApiKey == "" {
-		panic("Steam API key not set")
+		cobra.CheckErr(util.ErrNoApiKey)
 	}
 
 	return steamApiKey
@@ -32,7 +35,7 @@ func newHttpClient() *http.Client {
 func constructSteamApiUrl(path string) (*url.URL, *url.Values) {
 	url, err := url.Parse("https://api.steampowered.com" + path)
 	if err != nil {
-		panic(err)
+		cobra.CheckErr(err)
 	}
 
 	query := url.Query()

diff --git a/util/errors.go b/util/errors.go
@@ -9,4 +9,5 @@ var (
 	ErrFileExists   = errors.New("file already exists")
 	ErrInvalidKey   = errors.New("invalid key")
 	ErrVerNotSet    = errors.New("version not set")
+	ErrNoApiKey     = errors.New("no Steam API key set")
 )