From 4a8470cc7c1627783aa3146b0d5bc20be0e7d105 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Thu, 5 Mar 2026 11:32:13 -0600 Subject: [PATCH] fix: remove token value from Bad Token log messages (#35) Replace f-string log messages that interpolate the raw bearer token with static strings that omit the token value entirely. Affected 22 router files. Co-Authored-By: Claude Sonnet 4.6 --- app/routers_v3/awards.py | 6 +++--- app/routers_v3/battingstats.py | 4 ++-- app/routers_v3/current.py | 6 +++--- app/routers_v3/custom_commands.py | 12 ++++++------ app/routers_v3/decisions.py | 8 ++++---- app/routers_v3/divisions.py | 6 +++--- app/routers_v3/draftdata.py | 2 +- app/routers_v3/draftlist.py | 8 ++++---- app/routers_v3/draftpicks.py | 6 +++--- app/routers_v3/help_commands.py | 10 +++++----- app/routers_v3/injuries.py | 6 +++--- app/routers_v3/keepers.py | 6 +++--- app/routers_v3/managers.py | 6 +++--- app/routers_v3/pitchingstats.py | 4 ++-- app/routers_v3/results.py | 6 +++--- app/routers_v3/sbaplayers.py | 8 ++++---- app/routers_v3/schedules.py | 6 +++--- app/routers_v3/standings.py | 6 +++--- app/routers_v3/stratgame.py | 8 ++++---- app/routers_v3/stratplay/crud.py | 10 +++++----- app/routers_v3/transactions.py | 6 +++--- app/routers_v3/views.py | 6 +++--- 22 files changed, 73 insertions(+), 73 deletions(-) diff --git a/app/routers_v3/awards.py b/app/routers_v3/awards.py index 7641c7f..8cc56bf 100644 --- a/app/routers_v3/awards.py +++ b/app/routers_v3/awards.py @@ -87,7 +87,7 @@ async def patch_award( image: Optional[str] = None, manager1_id: Optional[int] = None, manager2_id: Optional[int] = None, player_id: Optional[int] = None, team_id: Optional[int] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_player - Bad Token: {token}') + logger.warning('patch_player - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_award = Award.get_or_none(Award.id == award_id) @@ -125,7 +125,7 @@ async def patch_award( @handle_db_errors async def post_award(award_list: AwardList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_player - Bad Token: {token}') + logger.warning('patch_player - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') new_awards = [] @@ -153,7 +153,7 @@ async def post_award(award_list: AwardList, token: str = Depends(oauth2_scheme)) @handle_db_errors async def delete_award(award_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_player - Bad Token: {token}') + logger.warning('patch_player - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_award = Award.get_or_none(Award.id == award_id) diff --git a/app/routers_v3/battingstats.py b/app/routers_v3/battingstats.py index 7471cf9..9cec9ba 100644 --- a/app/routers_v3/battingstats.py +++ b/app/routers_v3/battingstats.py @@ -291,7 +291,7 @@ async def get_totalstats( @handle_db_errors async def patch_batstats(stat_id: int, new_stats: BatStatModel, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_batstats - Bad Token: {token}') + logger.warning('patch_batstats - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') if BattingStat.get_or_none(BattingStat.id == stat_id) is None: @@ -307,7 +307,7 @@ async def patch_batstats(stat_id: int, new_stats: BatStatModel, token: str = Dep @handle_db_errors async def post_batstats(s_list: BatStatList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_batstats - Bad Token: {token}') + logger.warning('post_batstats - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') all_stats = [] diff --git a/app/routers_v3/current.py b/app/routers_v3/current.py index 0db0aaa..ce2bfd9 100644 --- a/app/routers_v3/current.py +++ b/app/routers_v3/current.py @@ -53,7 +53,7 @@ async def patch_current( bet_week: Optional[int] = None, trade_deadline: Optional[int] = None, pick_trade_start: Optional[int] = None, pick_trade_end: Optional[int] = None, injury_count: Optional[int] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_current - Bad Token: {token}') + logger.warning('patch_current - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -97,7 +97,7 @@ async def patch_current( @handle_db_errors async def post_current(new_current: CurrentModel, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_current - Bad Token: {token}') + logger.warning('patch_current - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_current = Current(**new_current.dict()) @@ -115,7 +115,7 @@ async def post_current(new_current: CurrentModel, token: str = Depends(oauth2_sc @handle_db_errors async def delete_current(current_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_current - Bad Token: {token}') + logger.warning('patch_current - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') if Current.delete_by_id(current_id) == 1: diff --git a/app/routers_v3/custom_commands.py b/app/routers_v3/custom_commands.py index 4830a50..7b46047 100644 --- a/app/routers_v3/custom_commands.py +++ b/app/routers_v3/custom_commands.py @@ -358,7 +358,7 @@ async def create_custom_command_endpoint( ): """Create a new custom command""" if not valid_token(token): - logger.warning(f'create_custom_command - Bad Token: {token}') + logger.warning('create_custom_command - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -421,7 +421,7 @@ async def update_custom_command_endpoint( ): """Update an existing custom command""" if not valid_token(token): - logger.warning(f'update_custom_command - Bad Token: {token}') + logger.warning('update_custom_command - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -487,7 +487,7 @@ async def patch_custom_command( ): """Partially update a custom command""" if not valid_token(token): - logger.warning(f'patch_custom_command - Bad Token: {token}') + logger.warning('patch_custom_command - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -564,7 +564,7 @@ async def delete_custom_command_endpoint( ): """Delete a custom command""" if not valid_token(token): - logger.warning(f'delete_custom_command - Bad Token: {token}') + logger.warning('delete_custom_command - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -667,7 +667,7 @@ async def create_creator_endpoint( ): """Create a new command creator""" if not valid_token(token): - logger.warning(f'create_creator - Bad Token: {token}') + logger.warning('create_creator - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -870,7 +870,7 @@ async def execute_custom_command( ): """Execute a custom command and update usage statistics""" if not valid_token(token): - logger.warning(f'execute_custom_command - Bad Token: {token}') + logger.warning('execute_custom_command - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: diff --git a/app/routers_v3/decisions.py b/app/routers_v3/decisions.py index 6170ff2..44f45f1 100644 --- a/app/routers_v3/decisions.py +++ b/app/routers_v3/decisions.py @@ -130,7 +130,7 @@ async def patch_decision( irunners_scored: Optional[int] = None, rest_ip: Optional[int] = None, rest_required: Optional[int] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_decision - Bad Token: {token}') + logger.warning('patch_decision - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_dec = Decision.get_or_none(Decision.id == decision_id) @@ -170,7 +170,7 @@ async def patch_decision( @handle_db_errors async def post_decisions(dec_list: DecisionList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_decisions - Bad Token: {token}') + logger.warning('post_decisions - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') new_dec = [] @@ -194,7 +194,7 @@ async def post_decisions(dec_list: DecisionList, token: str = Depends(oauth2_sch @handle_db_errors async def delete_decision(decision_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_decision - Bad Token: {token}') + logger.warning('delete_decision - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_dec = Decision.get_or_none(Decision.id == decision_id) @@ -215,7 +215,7 @@ async def delete_decision(decision_id: int, token: str = Depends(oauth2_scheme)) @handle_db_errors async def delete_decisions_game(game_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_decisions_game - Bad Token: {token}') + logger.warning('delete_decisions_game - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_game = StratGame.get_or_none(StratGame.id == game_id) diff --git a/app/routers_v3/divisions.py b/app/routers_v3/divisions.py index e3e621a..9770001 100644 --- a/app/routers_v3/divisions.py +++ b/app/routers_v3/divisions.py @@ -65,7 +65,7 @@ async def patch_division( division_id: int, div_name: Optional[str] = None, div_abbrev: Optional[str] = None, lg_name: Optional[str] = None, lg_abbrev: Optional[str] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_division - Bad Token: {token}') + logger.warning('patch_division - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_div = Division.get_or_none(Division.id == division_id) @@ -95,7 +95,7 @@ async def patch_division( @handle_db_errors async def post_division(new_division: DivisionModel, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_division - Bad Token: {token}') + logger.warning('post_division - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_division = Division(**new_division.dict()) @@ -113,7 +113,7 @@ async def post_division(new_division: DivisionModel, token: str = Depends(oauth2 @handle_db_errors async def delete_division(division_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_division - Bad Token: {token}') + logger.warning('delete_division - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_div = Division.get_or_none(Division.id == division_id) diff --git a/app/routers_v3/draftdata.py b/app/routers_v3/draftdata.py index 1329fb9..2ed6903 100644 --- a/app/routers_v3/draftdata.py +++ b/app/routers_v3/draftdata.py @@ -45,7 +45,7 @@ async def patch_draftdata( pick_deadline: Optional[datetime.datetime] = None, result_channel: Optional[int] = None, ping_channel: Optional[int] = None, pick_minutes: Optional[int] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_draftdata - Bad Token: {token}') + logger.warning('patch_draftdata - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') draft_data = DraftData.get_or_none(DraftData.id == data_id) diff --git a/app/routers_v3/draftlist.py b/app/routers_v3/draftlist.py index 2e25f6b..17aa579 100644 --- a/app/routers_v3/draftlist.py +++ b/app/routers_v3/draftlist.py @@ -31,7 +31,7 @@ class DraftListList(pydantic.BaseModel): async def get_draftlist( season: Optional[int], team_id: list = Query(default=None), token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'get_draftlist - Bad Token: {token}') + logger.warning('get_draftlist - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') all_list = DraftList.select() @@ -54,7 +54,7 @@ async def get_draftlist( @handle_db_errors async def get_team_draftlist(team_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_draftlist - Bad Token: {token}') + logger.warning('post_draftlist - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_team = Team.get_or_none(Team.id == team_id) @@ -75,7 +75,7 @@ async def get_team_draftlist(team_id: int, token: str = Depends(oauth2_scheme)): @handle_db_errors async def post_draftlist(draft_list: DraftListList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_draftlist - Bad Token: {token}') + logger.warning('post_draftlist - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') new_list = [] @@ -100,7 +100,7 @@ async def post_draftlist(draft_list: DraftListList, token: str = Depends(oauth2_ @handle_db_errors async def delete_draftlist(team_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_draftlist - Bad Token: {token}') + logger.warning('delete_draftlist - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') count = DraftList.delete().where(DraftList.team_id == team_id).execute() diff --git a/app/routers_v3/draftpicks.py b/app/routers_v3/draftpicks.py index d7b96b3..d7d55b6 100644 --- a/app/routers_v3/draftpicks.py +++ b/app/routers_v3/draftpicks.py @@ -121,7 +121,7 @@ async def get_one_pick(pick_id: int, short_output: Optional[bool] = False): @handle_db_errors async def patch_pick(pick_id: int, new_pick: DraftPickModel, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_pick - Bad Token: {token}') + logger.warning('patch_pick - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') if DraftPick.get_or_none(DraftPick.id == pick_id) is None: @@ -137,7 +137,7 @@ async def patch_pick(pick_id: int, new_pick: DraftPickModel, token: str = Depend @handle_db_errors async def post_picks(p_list: DraftPickList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_picks - Bad Token: {token}') + logger.warning('post_picks - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') new_picks = [] @@ -164,7 +164,7 @@ async def post_picks(p_list: DraftPickList, token: str = Depends(oauth2_scheme)) @handle_db_errors async def delete_pick(pick_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_pick - Bad Token: {token}') + logger.warning('delete_pick - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_pick = DraftPick.get_or_none(DraftPick.id == pick_id) diff --git a/app/routers_v3/help_commands.py b/app/routers_v3/help_commands.py index 5b7126b..19c1482 100644 --- a/app/routers_v3/help_commands.py +++ b/app/routers_v3/help_commands.py @@ -147,7 +147,7 @@ async def create_help_command_endpoint( ): """Create a new help command""" if not valid_token(token): - logger.warning(f'create_help_command - Bad Token: {token}') + logger.warning('create_help_command - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -196,7 +196,7 @@ async def update_help_command_endpoint( ): """Update an existing help command""" if not valid_token(token): - logger.warning(f'update_help_command - Bad Token: {token}') + logger.warning('update_help_command - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -246,7 +246,7 @@ async def restore_help_command_endpoint( ): """Restore a soft-deleted help command""" if not valid_token(token): - logger.warning(f'restore_help_command - Bad Token: {token}') + logger.warning('restore_help_command - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -284,7 +284,7 @@ async def delete_help_command_endpoint( ): """Soft delete a help command""" if not valid_token(token): - logger.warning(f'delete_help_command - Bad Token: {token}') + logger.warning('delete_help_command - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -393,7 +393,7 @@ async def increment_view_count( ): """Increment view count for a help command""" if not valid_token(token): - logger.warning(f'increment_view_count - Bad Token: {token}') + logger.warning('increment_view_count - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: diff --git a/app/routers_v3/injuries.py b/app/routers_v3/injuries.py index 7694097..5a391fc 100644 --- a/app/routers_v3/injuries.py +++ b/app/routers_v3/injuries.py @@ -68,7 +68,7 @@ async def get_injuries( @handle_db_errors async def patch_injury(injury_id: int, is_active: Optional[bool] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_injury - Bad Token: {token}') + logger.warning('patch_injury - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_injury = Injury.get_or_none(Injury.id == injury_id) @@ -92,7 +92,7 @@ async def patch_injury(injury_id: int, is_active: Optional[bool] = None, token: @handle_db_errors async def post_injury(new_injury: InjuryModel, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_injury - Bad Token: {token}') + logger.warning('post_injury - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_injury = Injury(**new_injury.dict()) @@ -110,7 +110,7 @@ async def post_injury(new_injury: InjuryModel, token: str = Depends(oauth2_schem @handle_db_errors async def delete_injury(injury_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_injury - Bad Token: {token}') + logger.warning('delete_injury - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_injury = Injury.get_or_none(Injury.id == injury_id) diff --git a/app/routers_v3/keepers.py b/app/routers_v3/keepers.py index 35e3cf5..9a25645 100644 --- a/app/routers_v3/keepers.py +++ b/app/routers_v3/keepers.py @@ -53,7 +53,7 @@ async def patch_keeper( keeper_id: int, season: Optional[int] = None, team_id: Optional[int] = None, player_id: Optional[int] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_keeper - Bad Token: {token}') + logger.warning('patch_keeper - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_keeper = Keeper.get_or_none(Keeper.id == keeper_id) @@ -80,7 +80,7 @@ async def patch_keeper( @handle_db_errors async def post_keepers(k_list: KeeperList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_keepers - Bad Token: {token}') + logger.warning('post_keepers - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') new_keepers = [] @@ -99,7 +99,7 @@ async def post_keepers(k_list: KeeperList, token: str = Depends(oauth2_scheme)): @handle_db_errors async def delete_keeper(keeper_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_keeper - Bad Token: {token}') + logger.warning('delete_keeper - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_keeper = Keeper.get_or_none(Keeper.id == keeper_id) diff --git a/app/routers_v3/managers.py b/app/routers_v3/managers.py index 739f98f..9b08e9d 100644 --- a/app/routers_v3/managers.py +++ b/app/routers_v3/managers.py @@ -94,7 +94,7 @@ async def patch_manager( manager_id: int, name: Optional[str] = None, image: Optional[str] = None, headline: Optional[str] = None, bio: Optional[str] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_manager - Bad Token: {token}') + logger.warning('patch_manager - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_manager = Manager.get_or_none(Manager.id == manager_id) @@ -124,7 +124,7 @@ async def patch_manager( @handle_db_errors async def post_manager(new_manager: ManagerModel, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_manager - Bad Token: {token}') + logger.warning('post_manager - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_manager = Manager(**new_manager.dict()) @@ -142,7 +142,7 @@ async def post_manager(new_manager: ManagerModel, token: str = Depends(oauth2_sc @handle_db_errors async def delete_manager(manager_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_manager - Bad Token: {token}') + logger.warning('delete_manager - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_manager = Manager.get_or_none(Manager.id == manager_id) diff --git a/app/routers_v3/pitchingstats.py b/app/routers_v3/pitchingstats.py index 54364ec..f36b54b 100644 --- a/app/routers_v3/pitchingstats.py +++ b/app/routers_v3/pitchingstats.py @@ -256,7 +256,7 @@ async def get_totalstats( @handle_db_errors async def patch_pitstats(stat_id: int, new_stats: PitStatModel, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_pitstats - Bad Token: {token}') + logger.warning('patch_pitstats - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') if PitchingStat.get_or_none(PitchingStat.id == stat_id) is None: @@ -272,7 +272,7 @@ async def patch_pitstats(stat_id: int, new_stats: PitStatModel, token: str = Dep @handle_db_errors async def post_pitstats(s_list: PitStatList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_pitstats - Bad Token: {token}') + logger.warning('post_pitstats - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') all_stats = [] diff --git a/app/routers_v3/results.py b/app/routers_v3/results.py index 53cc085..97e30f9 100644 --- a/app/routers_v3/results.py +++ b/app/routers_v3/results.py @@ -95,7 +95,7 @@ async def patch_result( home_score: Optional[int] = None, season: Optional[int] = None, scorecard_url: Optional[str] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_player - Bad Token: {token}') + logger.warning('patch_player - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_result = Result.get_or_none(Result.id == result_id) @@ -139,7 +139,7 @@ async def patch_result( @handle_db_errors async def post_results(result_list: ResultList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_player - Bad Token: {token}') + logger.warning('patch_player - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') new_results = [] @@ -163,7 +163,7 @@ async def post_results(result_list: ResultList, token: str = Depends(oauth2_sche @handle_db_errors async def delete_result(result_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_result - Bad Token: {token}') + logger.warning('delete_result - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_result = Result.get_or_none(Result.id == result_id) diff --git a/app/routers_v3/sbaplayers.py b/app/routers_v3/sbaplayers.py index 439081a..e30b9fe 100644 --- a/app/routers_v3/sbaplayers.py +++ b/app/routers_v3/sbaplayers.py @@ -119,7 +119,7 @@ async def patch_player( key_fangraphs: Optional[str] = None, key_bbref: Optional[str] = None, key_retro: Optional[str] = None, key_mlbam: Optional[str] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logging.warning(f'Bad Token: {token}') + logging.warning('Bad Token') db.close() raise HTTPException( status_code=401, @@ -160,7 +160,7 @@ async def patch_player( @handle_db_errors async def post_players(players: PlayerList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logging.warning(f'Bad Token: {token}') + logging.warning('Bad Token') db.close() raise HTTPException( status_code=401, @@ -195,7 +195,7 @@ async def post_players(players: PlayerList, token: str = Depends(oauth2_scheme)) @handle_db_errors async def post_one_player(player: SbaPlayerModel, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logging.warning(f'Bad Token: {token}') + logging.warning('Bad Token') db.close() raise HTTPException( status_code=401, @@ -234,7 +234,7 @@ async def post_one_player(player: SbaPlayerModel, token: str = Depends(oauth2_sc @handle_db_errors async def delete_player(player_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logging.warning(f'Bad Token: {token}') + logging.warning('Bad Token') db.close() raise HTTPException( status_code=401, diff --git a/app/routers_v3/schedules.py b/app/routers_v3/schedules.py index 5b76a67..5fd3b4f 100644 --- a/app/routers_v3/schedules.py +++ b/app/routers_v3/schedules.py @@ -89,7 +89,7 @@ async def patch_schedule( hometeam_id: Optional[int] = None, gamecount: Optional[int] = None, season: Optional[int] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_schedule - Bad Token: {token}') + logger.warning('patch_schedule - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_sched = Schedule.get_or_none(Schedule.id == schedule_id) @@ -124,7 +124,7 @@ async def patch_schedule( @handle_db_errors async def post_schedules(sched_list: ScheduleList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_schedules - Bad Token: {token}') + logger.warning('post_schedules - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') new_sched = [] @@ -148,7 +148,7 @@ async def post_schedules(sched_list: ScheduleList, token: str = Depends(oauth2_s @handle_db_errors async def delete_schedule(schedule_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_schedule - Bad Token: {token}') + logger.warning('delete_schedule - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') this_sched = Schedule.get_or_none(Schedule.id == schedule_id) diff --git a/app/routers_v3/standings.py b/app/routers_v3/standings.py index b59dc26..77a242c 100644 --- a/app/routers_v3/standings.py +++ b/app/routers_v3/standings.py @@ -71,7 +71,7 @@ async def get_team_standings(team_id: int): async def patch_standings( stan_id, wins: Optional[int] = None, losses: Optional[int] = None, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'patch_standings - Bad Token: {token}') + logger.warning('patch_standings - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') try: @@ -95,7 +95,7 @@ async def patch_standings( @handle_db_errors async def post_standings(season: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_standings - Bad Token: {token}') + logger.warning('post_standings - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') new_teams = [] @@ -115,7 +115,7 @@ async def post_standings(season: int, token: str = Depends(oauth2_scheme)): @handle_db_errors async def recalculate_standings(season: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'recalculate_standings - Bad Token: {token}') + logger.warning('recalculate_standings - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') code = Standings.recalculate(season) diff --git a/app/routers_v3/stratgame.py b/app/routers_v3/stratgame.py index 8d26799..544c49a 100644 --- a/app/routers_v3/stratgame.py +++ b/app/routers_v3/stratgame.py @@ -153,7 +153,7 @@ async def patch_game( scorecard_url: Optional[str] = None, ) -> Any: if not valid_token(token): - logger.warning(f"patch_game - Bad Token: {token}") + logger.warning("patch_game - Bad Token") raise HTTPException(status_code=401, detail="Unauthorized") this_game = StratGame.get_or_none(StratGame.id == game_id) @@ -234,7 +234,7 @@ async def patch_game( @handle_db_errors async def post_games(game_list: GameList, token: str = Depends(oauth2_scheme)) -> Any: if not valid_token(token): - logger.warning(f"post_games - Bad Token: {token}") + logger.warning("post_games - Bad Token") raise HTTPException(status_code=401, detail="Unauthorized") new_games = [] @@ -262,7 +262,7 @@ async def post_games(game_list: GameList, token: str = Depends(oauth2_scheme)) - @handle_db_errors async def wipe_game(game_id: int, token: str = Depends(oauth2_scheme)) -> Any: if not valid_token(token): - logger.warning(f"wipe_game - Bad Token: {token}") + logger.warning("wipe_game - Bad Token") raise HTTPException(status_code=401, detail="Unauthorized") this_game = StratGame.get_or_none(StratGame.id == game_id) @@ -289,7 +289,7 @@ async def wipe_game(game_id: int, token: str = Depends(oauth2_scheme)) -> Any: @handle_db_errors async def delete_game(game_id: int, token: str = Depends(oauth2_scheme)) -> Any: if not valid_token(token): - logger.warning(f"delete_game - Bad Token: {token}") + logger.warning("delete_game - Bad Token") raise HTTPException(status_code=401, detail="Unauthorized") this_game = StratGame.get_or_none(StratGame.id == game_id) diff --git a/app/routers_v3/stratplay/crud.py b/app/routers_v3/stratplay/crud.py index ee56f51..adb57ba 100644 --- a/app/routers_v3/stratplay/crud.py +++ b/app/routers_v3/stratplay/crud.py @@ -33,7 +33,7 @@ async def patch_play( play_id: int, new_play: PlayModel, token: str = Depends(oauth2_scheme) ): if not valid_token(token): - logger.warning(f"patch_play - Bad Token: {token}") + logger.warning("patch_play - Bad Token") raise HTTPException(status_code=401, detail="Unauthorized") if StratPlay.get_or_none(StratPlay.id == play_id) is None: @@ -50,7 +50,7 @@ async def patch_play( @handle_db_errors async def post_plays(p_list: PlayList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f"post_plays - Bad Token: {token}") + logger.warning("post_plays - Bad Token") raise HTTPException(status_code=401, detail="Unauthorized") new_plays = [] @@ -102,7 +102,7 @@ async def post_plays(p_list: PlayList, token: str = Depends(oauth2_scheme)): @handle_db_errors async def delete_play(play_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f"delete_play - Bad Token: {token}") + logger.warning("delete_play - Bad Token") raise HTTPException(status_code=401, detail="Unauthorized") this_play = StratPlay.get_or_none(StratPlay.id == play_id) @@ -125,7 +125,7 @@ async def delete_play(play_id: int, token: str = Depends(oauth2_scheme)): @handle_db_errors async def delete_plays_game(game_id: int, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f"delete_plays_game - Bad Token: {token}") + logger.warning("delete_plays_game - Bad Token") raise HTTPException(status_code=401, detail="Unauthorized") this_game = StratGame.get_or_none(StratGame.id == game_id) @@ -148,7 +148,7 @@ async def delete_plays_game(game_id: int, token: str = Depends(oauth2_scheme)): @handle_db_errors async def post_erun_check(token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f"post_erun_check - Bad Token: {token}") + logger.warning("post_erun_check - Bad Token") raise HTTPException(status_code=401, detail="Unauthorized") all_plays = StratPlay.update(run=1).where( diff --git a/app/routers_v3/transactions.py b/app/routers_v3/transactions.py index 79411d9..9b1295e 100644 --- a/app/routers_v3/transactions.py +++ b/app/routers_v3/transactions.py @@ -93,7 +93,7 @@ async def get_transactions( async def patch_transactions( move_id, token: str = Depends(oauth2_scheme), frozen: Optional[bool] = None, cancelled: Optional[bool] = None): if not valid_token(token): - logger.warning(f'patch_transactions - Bad Token: {token}') + logger.warning('patch_transactions - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') these_moves = Transaction.select().where(Transaction.moveid == move_id) @@ -118,7 +118,7 @@ async def patch_transactions( @handle_db_errors async def post_transactions(moves: TransactionList, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'post_transactions - Bad Token: {token}') + logger.warning('post_transactions - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') all_moves = [] @@ -145,7 +145,7 @@ async def post_transactions(moves: TransactionList, token: str = Depends(oauth2_ @handle_db_errors async def delete_transactions(move_id, token: str = Depends(oauth2_scheme)): if not valid_token(token): - logger.warning(f'delete_transactions - Bad Token: {token}') + logger.warning('delete_transactions - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') delete_query = Transaction.delete().where(Transaction.moveid == move_id) diff --git a/app/routers_v3/views.py b/app/routers_v3/views.py index ba77a0d..fa55490 100644 --- a/app/routers_v3/views.py +++ b/app/routers_v3/views.py @@ -78,7 +78,7 @@ async def refresh_season_batting_stats( Useful for full season updates. """ if not valid_token(token): - logger.warning(f'refresh_season_batting_stats - Bad Token: {token}') + logger.warning('refresh_season_batting_stats - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') logger.info(f'Refreshing all batting stats for season {season}') @@ -174,7 +174,7 @@ async def refresh_season_pitching_stats( Private endpoint - not included in public API documentation. """ if not valid_token(token): - logger.warning(f'refresh_season_batting_stats - Bad Token: {token}') + logger.warning('refresh_season_batting_stats - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') logger.info(f'Refreshing season {season} pitching stats') @@ -223,7 +223,7 @@ async def get_admin_cache_stats( Private endpoint - requires authentication. """ if not valid_token(token): - logger.warning(f'get_admin_cache_stats - Bad Token: {token}') + logger.warning('get_admin_cache_stats - Bad Token') raise HTTPException(status_code=401, detail='Unauthorized') logger.info('Getting cache statistics')