From 6a456dc6b7271fae34d6a1a71a4b74999e724d2f Mon Sep 17 00:00:00 2001 From: Anass Bouassaba Date: Fri, 6 Sep 2024 14:23:00 +0200 Subject: [PATCH] fix(api): avoid division by zero in list size --- api/router/file_router.go | 3 +++ api/router/group_router.go | 3 +++ api/router/insights_router.go | 3 +++ api/router/invitation_router.go | 6 ++++++ api/router/organization_router.go | 3 +++ api/router/snapshot_router.go | 3 +++ api/router/task_router.go | 3 +++ api/router/user_router.go | 3 +++ api/router/workspace_router.go | 3 +++ 9 files changed, 30 insertions(+) diff --git a/api/router/file_router.go b/api/router/file_router.go index f806b8519..c3c1cb482 100644 --- a/api/router/file_router.go +++ b/api/router/file_router.go @@ -360,6 +360,9 @@ func (r *FileRouter) List(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by") diff --git a/api/router/group_router.go b/api/router/group_router.go index 0ea67be90..9017925c7 100644 --- a/api/router/group_router.go +++ b/api/router/group_router.go @@ -129,6 +129,9 @@ func (r *GroupRouter) List(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by") diff --git a/api/router/insights_router.go b/api/router/insights_router.go index c514985b1..591808790 100644 --- a/api/router/insights_router.go +++ b/api/router/insights_router.go @@ -183,6 +183,9 @@ func (r *InsightsRouter) ListEntities(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by") diff --git a/api/router/invitation_router.go b/api/router/invitation_router.go index db7e1a59e..055b9bd02 100644 --- a/api/router/invitation_router.go +++ b/api/router/invitation_router.go @@ -107,6 +107,9 @@ func (r *InvitationRouter) GetIncoming(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by") @@ -184,6 +187,9 @@ func (r *InvitationRouter) GetOutgoing(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by") diff --git a/api/router/organization_router.go b/api/router/organization_router.go index 6f3ad3bf3..93433e4b4 100644 --- a/api/router/organization_router.go +++ b/api/router/organization_router.go @@ -187,6 +187,9 @@ func (r *OrganizationRouter) List(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by") diff --git a/api/router/snapshot_router.go b/api/router/snapshot_router.go index c3f1bc33a..e215a0654 100644 --- a/api/router/snapshot_router.go +++ b/api/router/snapshot_router.go @@ -84,6 +84,9 @@ func (r *SnapshotRouter) List(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by") diff --git a/api/router/task_router.go b/api/router/task_router.go index 075f53033..9767933ea 100644 --- a/api/router/task_router.go +++ b/api/router/task_router.go @@ -106,6 +106,9 @@ func (r *TaskRouter) List(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by") diff --git a/api/router/user_router.go b/api/router/user_router.go index 5117d2499..5364a1bd2 100644 --- a/api/router/user_router.go +++ b/api/router/user_router.go @@ -72,6 +72,9 @@ func (r *UserRouter) List(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by") diff --git a/api/router/workspace_router.go b/api/router/workspace_router.go index ba6146dc9..667616524 100644 --- a/api/router/workspace_router.go +++ b/api/router/workspace_router.go @@ -126,6 +126,9 @@ func (r *WorkspaceRouter) List(c *fiber.Ctx) error { return err } } + if size == 0 { + return errorpkg.NewInvalidQueryParamError("size") + } sortBy := c.Query("sort_by") if !IsValidSortBy(sortBy) { return errorpkg.NewInvalidQueryParamError("sort_by")