Less obtuse error handling from API responses

api/api.go

@@ -2,8 +2,8 @@ package api
 
 import (
 	"bytes"
+	"encoding/json"
 	"fmt"
-	"github.com/buger/jsonparser"
 	"github.com/pkg/errors"
 	"github.com/pterodactyl/wings/config"
 	"go.uber.org/zap"
@@ -24,12 +24,6 @@ type PanelRequest struct {
 	Response *http.Response
 }
 
-func IsRequestError (err error) bool {
-	_, ok := err.(*PanelRequest)
-
-	return ok
-}
-
 // Builds the base request instance that can be used with the HTTP client.
 func (r *PanelRequest) GetClient() *http.Client {
 	return &http.Client{Timeout: time.Second * 30}
@@ -116,29 +110,34 @@ func (r *PanelRequest) HttpResponseCode() int {
 	return r.Response.StatusCode
 }
 
+type RequestError struct {
+	Code   string `json:"code"`
+	Status string `json:"status"`
+	Detail string `json:"detail"`
+}
+
+// Returns the error response in a string form that can be more easily consumed.
+func (re *RequestError) String() string {
+	return fmt.Sprintf("%s: %s (HTTP/%s)", re.Code, re.Detail, re.Status)
+}
+
+type RequestErrorBag struct {
+	Errors []RequestError `json:"errors"`
+}
+
 // Returns the error message from the API call as a string. The error message will be formatted
 // similar to the below example:
 //
 // HttpNotFoundException: The requested resource does not exist. (HTTP/404)
-func (r *PanelRequest) Error() string {
+func (r *PanelRequest) Error() *RequestError {
-	body, err := r.ReadBody()
+	body, _ := r.ReadBody()
-	if err != nil {
+
-		return err.Error()
+	bag := RequestErrorBag{}
+	json.Unmarshal(body, &bag)
+
+	if len(bag.Errors) == 0 {
+		return new(RequestError)
 	}
 
-	zap.S().Debugw("got body", zap.ByteString("b", body))
+	return &bag.Errors[0]
-	_, valueType, _, err := jsonparser.Get(body, "errors")
-	if err != nil {
-		return err.Error()
-	}
-
-	if valueType != jsonparser.Object {
-		return "no error object present on response"
-	}
-
-	code, _ := jsonparser.GetString(body, "errors.0.code")
-	status, _ := jsonparser.GetString(body, "errors.0.status")
-	detail, _ := jsonparser.GetString(body, "errors.0.detail")
-
-	return fmt.Sprintf("%s: %s (HTTP/%s)", code, detail, status)
 }

api/server_endpoints.go

@@ -5,7 +5,6 @@ import (
 	"fmt"
 	"github.com/pkg/errors"
 	"github.com/pterodactyl/wings/parser"
-	"go.uber.org/zap"
 )
 
 const (
@@ -40,9 +39,7 @@ func (r *PanelRequest) GetServerConfiguration(uuid string) (*ServerConfiguration
 	r.Response = resp
 
 	if r.HasError() {
-		zap.S().Warnw("got error", zap.String("message", r.Error()))
+		return nil, errors.WithStack(errors.New(r.Error().String()))
-
-		return nil, errors.WithStack(errors.New(r.Error()))
 	}
 
 	res := &ServerConfiguration{}

api/sftp_endpoints.go

@@ -25,7 +25,7 @@ func (r *PanelRequest) ValidateSftpCredentials(request sftp_server.Authenticatio
 			return nil, sftp_server.InvalidCredentialsError{}
 		}
 
-		return nil, errors.WithStack(errors.New(r.Error()))
+		return nil, errors.WithStack(errors.New(r.Error().String()))
 	}
 
 	response := new(sftp_server.AuthenticationResponse)

server/server.go

@@ -173,13 +173,13 @@ func LoadDirectory(dir string, cfg *config.SystemConfiguration) error {
 
 			b, err := ioutil.ReadFile(path.Join(dir, file.Name()))
 			if err != nil {
-				zap.S().Errorw("failed to read server configuration file, skipping...", zap.Error(err))
+				zap.S().Errorw("failed to read server configuration file, skipping...", zap.String("server", file.Name()), zap.Error(err))
 				return
 			}
 
 			s, err := FromConfiguration(b, cfg)
 			if err != nil {
-				zap.S().Errorw("failed to parse server configuration, skipping...", zap.Error(err))
+				zap.S().Errorw("failed to parse server configuration, skipping...", zap.String("server", file.Name()), zap.Error(err))
 				return
 			}