github/AutoGPT
1
1
Fork 0
mirror of https://github.com/Significant-Gravitas/AutoGPT.git synced 2026-04-08 03:00:28 -04:00
Code Issues Packages Projects Releases Wiki Activity

fix(platform): use lazy logger formatting in rate limit admin routes

Browse Source 
Replace f-string interpolation in logger.info() calls with %s-style
lazy formatting to avoid unnecessary string construction when the log
level is above INFO.
This commit is contained in:
Zamil Majdy
2026-03-29 06:42:03 +02:00
parent dc272559c6
commit c7430eaffb
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
autogpt_platform/backend/backend/api/features/admin/rate_limit_admin_routes.py
View File

@@ -173,7 +173,7 @@ async def get_user_rate_limit_tier(
admin_user_id: str = Security(get_user_id),
) -> UserTierResponse:
"""Get a user's current rate-limit tier. Admin-only."""
logger.info(f"Admin {admin_user_id} checking tier for user {user_id}")
logger.info("Admin %s checking tier for user %s", admin_user_id, user_id)
tier = await get_user_tier(user_id)
return UserTierResponse(user_id=user_id, tier=tier)
@@ -189,8 +189,10 @@ async def set_user_rate_limit_tier(
) -> UserTierResponse:
"""Set a user's rate-limit tier. Admin-only."""
logger.info(
f"Admin {admin_user_id} setting tier for user {request.user_id} "
f"to {request.tier.value}"
"Admin %s setting tier for user %s to %s",
admin_user_id,
request.user_id,
request.tier.value,
)
try:
await set_user_tier(request.user_id, request.tier)