cal/major-domo-database
1
0
Fork 0
Code Issues 25 Pull Requests 25 Actions Packages Projects Releases Wiki Activity

fix: enforce Literal validation on sort parameter in GET /api/v3/players (#66) #68

Open
Claude wants to merge 1 commits from ai/major-domo-database-66 into main
pull from: ai/major-domo-database-66
merge into: cal:main
Conversation 0 Commits 1 Files Changed 1 +2 -2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/routers_v3/players.py
View File

@ -4,7 +4,7 @@ Thin HTTP layer using PlayerService for business logic.
""" """
from fastapi import APIRouter, Query, Response, Depends from fastapi import APIRouter, Query, Response, Depends
from typing import Optional, List from typing import Literal, Optional, List
from ..dependencies import oauth2_scheme, cache_result, handle_db_errors from ..dependencies import oauth2_scheme, cache_result, handle_db_errors
from ..services.base import BaseService from ..services.base import BaseService
@ -23,7 +23,7 @@ async def get_players(
pos: list = Query(default=None), pos: list = Query(default=None),
strat_code: list = Query(default=None), strat_code: list = Query(default=None),
is_injured: Optional[bool] = None, is_injured: Optional[bool] = None,
sort: Optional[str] = None, sort: Optional[Literal["cost-asc", "cost-desc", "name-asc", "name-desc"]] = None,
limit: Optional[int] = Query( limit: Optional[int] = Query(
default=None, ge=1, description="Maximum number of results to return" default=None, ge=1, description="Maximum number of results to return"
), ),