From 2acb97145a08f0ff7f07f95ae077e868b5bb687d Mon Sep 17 00:00:00 2001
From: pg9182 <96569817+pg9182@users.noreply.github.com>
Date: Thu, 13 Oct 2022 11:53:30 -0400
Subject: pkg/api/api0: Better error messages for missing params

---
 pkg/api/api0/playerinfo.go | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

(limited to 'pkg/api/api0')

diff --git a/pkg/api/api0/playerinfo.go b/pkg/api/api0/playerinfo.go
index cacdbd6..b47ef4c 100644
--- a/pkg/api/api0/playerinfo.go
+++ b/pkg/api/api0/playerinfo.go
@@ -79,7 +79,17 @@ func (h *Handler) handlePlayer(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	uid, err := strconv.ParseUint(r.URL.Query().Get("id"), 10, 64)
+	uidQ := r.URL.Query().Get("id")
+	if uidQ == "" {
+		respJSON(w, r, http.StatusBadRequest, map[string]any{
+			"success": false,
+			"error":   ErrorCode_BAD_REQUEST,
+			"msg":     fmt.Sprintf("%s: id param is required", ErrorCode_BAD_REQUEST.Message()),
+		})
+		return
+	}
+
+	uid, err := strconv.ParseUint(uidQ, 10, 64)
 	if err != nil {
 		respJSON(w, r, http.StatusNotFound, map[string]any{
 			"success": false,
-- 
cgit v1.2.3