Merge pull request #2 from codetheuri/alert-autofix-2

Potential fix for code scanning alert no. 2: Incorrect conversion between integer types

Author: codetheuri

internal/app/handlers/todo.go

@@ -4,6 +4,7 @@ import (
 	"encoding/json"
 	"net/http"
 	"strconv"
+	"math"
 
 	"github.com/codetheuri/todolist/internal/app/services"
 	appErrors "github.com/codetheuri/todolist/pkg/errors"
@@ -63,6 +64,12 @@ func (h *TodoHandler) GetTodoByID(w http.ResponseWriter, r *http.Request) {
 		web.RespondError(w, appErrors.ValidationError("Invalid ID format", err, nil), http.StatusBadRequest)
 		return
 	}
+	// Check if the parsed ID is within the bounds of the uint type
+	if id > math.MaxUint {
+		h.log.Warn("Handler: ID exceeds the maximum allowed value for uint", "id", id)
+		web.RespondError(w, appErrors.ValidationError("ID exceeds the maximum allowed value", nil, nil), http.StatusBadRequest)
+		return
+	}
 	res, err := h.todoService.GetTodoByID(uint(id))
 	if err != nil {
 		h.log.Error("Handler: Service call failed for GetTodoByID", err, "todoID", id)