Skip to content

Commit 46e4666

Browse files
committed
fix: added query validation
1 parent 3da44fa commit 46e4666

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

  • diracx-routers/src/diracx/routers/pilots

diracx-routers/src/diracx/routers/pilots/query.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,10 @@
33
from http import HTTPStatus
44
from typing import Annotated, Any
55

6-
from fastapi import Body, Depends, Response
6+
from fastapi import Body, Depends, Query, Response
77

88
from diracx.core.models.search import SearchParams, SummaryParams
9+
from diracx.logic.pilots.query import MAX_PER_PAGE
910
from diracx.logic.pilots.query import search as search_bl
1011
from diracx.logic.pilots.query import summary as summary_bl
1112

@@ -111,8 +112,8 @@ async def search(
111112
check_permissions: CheckPilotManagementPolicyCallable,
112113
response: Response,
113114
user_info: Annotated[AuthorizedUserInfo, Depends(verify_dirac_access_token)],
114-
page: int = 1,
115-
per_page: int = 100,
115+
page: Annotated[int, Query(ge=1)] = 1,
116+
per_page: Annotated[int, Query(ge=1, le=MAX_PER_PAGE)] = 100,
116117
body: Annotated[
117118
SearchParams | None, Body(openapi_examples=EXAMPLE_SEARCHES) # type: ignore
118119
] = None,

0 commit comments

Comments
 (0)