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

fix: Prevent locals() from polluting player PATCH data dict #4

Merged
cal merged 1 commits from fix/locals-pollution-patch-endpoint into main 2026-02-04 21:00:24 +00:00
Conversation 0 Commits 1 Files Changed 2 +5 -4
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
VERSION
View File

@ -1 +1 @@
2.5.2 2.5.3

7
app/routers_v3/players.py
View File

@ -117,12 +117,13 @@ async def patch_player(
): ):
"""Patch a player (partial update).""" """Patch a player (partial update)."""
# Build dict of provided fields # Build dict of provided fields
data = {} # IMPORTANT: Capture locals() BEFORE creating data dict to avoid including 'data' itself
locals_dict = locals() locals_dict = locals()
data = {}
for key, value in locals_dict.items(): for key, value in locals_dict.items():
if key not in ('player_id', 'token') and value is not None: if key not in ('player_id', 'token', 'locals_dict') and value is not None:
data[key] = value data[key] = value
return PlayerService.patch_player(player_id, data, token) return PlayerService.patch_player(player_id, data, token)