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

fix req changes by greptile

Browse source
This commit is contained in:
Harshit28j 2026-03-20 23:57:00 +05:30 • committed by shivam
parent 4dec34b752
commit 69928449f7
2 changed files with 14 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
litellm/litellm_core_utils/litellm_logging.py
View file

@ -620,7 +620,7 @@ class Logging(LiteLLMLoggingBaseClass):
if "litellm_metadata" in kwargs and isinstance(
kwargs["litellm_metadata"], dict
):
if "metadata" not in base_litellm_params:
if not base_litellm_params.get("metadata"):
base_litellm_params["metadata"] = dict(kwargs["litellm_metadata"])
else:
base_litellm_params["metadata"] = dict(base_litellm_params["metadata"])

13
tests/test_litellm/litellm_core_utils/test_litellm_logging.py
View file

@ -467,6 +467,19 @@ class TestUpdateFromKwargs:
assert result_meta["from_caller"] is True
assert result_meta["user_api_key_hash"] == "hashed-xyz"
def test_merge_handles_metadata_none(self, logging_obj):
"""When metadata=None is explicitly in kwargs alongside litellm_metadata,
the merge should not crash (dict(None) would TypeError)."""
kwargs = {
"metadata": None,
"litellm_metadata": {"user_api_key_hash": "hashed-abc"},
}
logging_obj.update_from_kwargs(kwargs=kwargs)
result_meta = logging_obj.litellm_params["metadata"]
assert result_meta["user_api_key_hash"] == "hashed-abc"
def test_caller_litellm_params_win_over_kwargs(self, logging_obj):
"""Explicit litellm_params from the caller should override auto-extracted values."""
kwargs = {"metadata": {"from_kwargs": True}}