From d92f571960938ec6a581e75e4da5410fb70602a3 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Wed, 1 Apr 2026 20:14:35 -0500 Subject: [PATCH] hotfix: remove output caps from GET /players endpoint MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The MAX_LIMIT/DEFAULT_LIMIT caps added in 16f3f8d are too restrictive for the /players endpoint — bot and website consumers need full player lists without pagination. Reverts limit param to Optional[int] with no ceiling while keeping caps on all other endpoints. Co-Authored-By: Claude Opus 4.6 (1M context) --- app/routers_v3/players.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/app/routers_v3/players.py b/app/routers_v3/players.py index 13ac6f1..1801833 100644 --- a/app/routers_v3/players.py +++ b/app/routers_v3/players.py @@ -10,8 +10,6 @@ from ..dependencies import ( oauth2_scheme, cache_result, handle_db_errors, - MAX_LIMIT, - DEFAULT_LIMIT, ) from ..services.base import BaseService from ..services.player_service import PlayerService @@ -30,7 +28,7 @@ async def get_players( strat_code: list = Query(default=None), is_injured: Optional[bool] = None, sort: Optional[str] = None, - limit: int = Query(default=DEFAULT_LIMIT, ge=1, le=MAX_LIMIT), + limit: Optional[int] = Query(default=None, ge=1), offset: Optional[int] = Query( default=None, ge=0, description="Number of results to skip for pagination" ), -- 2.25.1