Skip to content

Commit 2958273

Browse files
committed
feat: add pagination limits to user and item list endpoints
1 parent 4d3d5e9 commit 2958273

2 files changed

Lines changed: 9 additions & 4 deletions

File tree

backend/app/api/routes/items.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import uuid
22
from typing import Any
33

4-
from fastapi import APIRouter, HTTPException
4+
from fastapi import APIRouter, HTTPException, Query
55
from sqlmodel import col, func, select
66

77
from app.api.deps import CurrentUser, SessionDep
@@ -12,7 +12,10 @@
1212

1313
@router.get("/", response_model=ItemsPublic)
1414
def read_items(
15-
session: SessionDep, current_user: CurrentUser, skip: int = 0, limit: int = 100
15+
session: SessionDep,
16+
current_user: CurrentUser,
17+
skip: int = 0,
18+
limit: int = Query(default=100, le=100),
1619
) -> Any:
1720
"""
1821
Retrieve items.

backend/app/api/routes/users.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import uuid
22
from typing import Any
33

4-
from fastapi import APIRouter, Depends, HTTPException
4+
from fastapi import APIRouter, Depends, HTTPException, Query
55
from sqlmodel import col, delete, func, select
66

77
from app import crud
@@ -34,7 +34,9 @@
3434
dependencies=[Depends(get_current_active_superuser)],
3535
response_model=UsersPublic,
3636
)
37-
def read_users(session: SessionDep, skip: int = 0, limit: int = 100) -> Any:
37+
def read_users(
38+
session: SessionDep, skip: int = 0, limit: int = Query(default=100, le=100)
39+
) -> Any:
3840
"""
3941
Retrieve users.
4042
"""

0 commit comments

Comments
 (0)