Avoid squashing http response err info with unmarshal err

Author: nephomaniac

Diff for: pkg/utils/util.go

@@ -105,19 +105,25 @@ func MatchBaseDomain(longHostname, baseDomain string) bool {
 }
 
 func TryParseBackplaneAPIError(rsp *http.Response) (*BackplaneApi.Error, error) {
+	if rsp == nil {
+		return nil, fmt.Errorf("parse err provided nil http response")
+	}
 	bodyBytes, err := io.ReadAll(rsp.Body)
-	defer func() { _ = rsp.Body.Close() }()
+	defer func() {
+		_ = rsp.Body.Close()
+	}()
 	if err != nil {
 		return nil, err
+	} else {
+		var dest BackplaneApi.Error
+		if err := json.Unmarshal(bodyBytes, &dest); err != nil {
+			// Avoid squashing the HTTP response info with Unmarshal err...
+			bodyStr := strings.ReplaceAll(string(bodyBytes[:]), "\n", " ")
+			err := fmt.Errorf("status:'%s', code:'%d'; failed to unmarshal response:'%s'; %w", rsp.Status, rsp.StatusCode, bodyStr, err)
+			return nil, err
+		}
+		return &dest, nil
 	}
-
-	var dest BackplaneApi.Error
-
-	if err := json.Unmarshal(bodyBytes, &dest); err != nil {
-		return nil, err
-	}
-
-	return &dest, nil
 }
 
 func TryRenderErrorRaw(rsp *http.Response) error {