scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-08 03:08:45 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 115

fix: guard cache_hit filter against non-string defaults in direct calls

Browse source 
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
yassin 2026-08-27 01:43:24 +00:00
parent 1cd1b9dabe
commit 87f50202c2
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
litellm/proxy/spend_tracking/spend_management_endpoints.py
View file

@ -2316,7 +2316,9 @@ async def ui_view_spend_logs(
param="sort_order",
code=status.HTTP_400_BAD_REQUEST,
)
if cache_hit is not None and cache_hit.lower() not in {"true", "false"}:
# isinstance guard: direct callers (tests) may leave the fastapi.Query default in place
cache_hit_filter: Final = cache_hit.lower() if isinstance(cache_hit, str) else None
if isinstance(cache_hit, str) and cache_hit_filter not in {"true", "false"}:
raise ProxyException(
message=f"Invalid cache_hit: {cache_hit}. Must be one of: true, false",
type="bad_request",
@ -2554,8 +2556,8 @@ async def ui_view_spend_logs(
sql_params.append(f"%{like_escaped_session_id}%")
p += 1
if cache_hit is not None:
if cache_hit.lower() == "true":
if cache_hit_filter is not None:
if cache_hit_filter == "true":
sql_conditions.append("LOWER(cache_hit) = 'true'")
else:
sql_conditions.append("(cache_hit IS NULL OR LOWER(cache_hit) <> 'true')")