mirror of
https://github.com/BerriAI/litellm.git
synced 2026-10-04 02:31:27 +00:00
fix(openai): keep ssl_verify out of provider params
This commit is contained in:
parent
340b1a2b2e
commit
4fa710628c
2 changed files with 14 additions and 0 deletions
|
|
@ -3791,6 +3791,7 @@ all_litellm_params = (
|
|||
"rust",
|
||||
"prompt_label",
|
||||
"shared_session",
|
||||
"ssl_verify",
|
||||
"search_tool_name",
|
||||
"order",
|
||||
"enable_tag_filtering",
|
||||
|
|
|
|||
|
|
@ -5268,6 +5268,19 @@ def test_client_side_timeout_marker_never_reaches_the_provider():
|
|||
)
|
||||
|
||||
|
||||
def test_ssl_verify_never_reaches_the_provider_params():
|
||||
"""SSL transport configuration must stay in litellm_params instead of extra_body."""
|
||||
kwargs = {"a_real_provider_specific_param": 1, "ssl_verify": "/tmp/ca.pem"}
|
||||
|
||||
non_default = get_non_default_completion_params(kwargs)
|
||||
|
||||
assert non_default == {"a_real_provider_specific_param": 1}, (
|
||||
"ssl_verify leaked into the provider params: "
|
||||
f"{sorted(set(non_default) - {'a_real_provider_specific_param'})}"
|
||||
)
|
||||
assert "ssl_verify" in all_litellm_params
|
||||
|
||||
|
||||
class _RecordingDeploymentFailureLogger(CustomLogger):
|
||||
def __init__(self) -> None:
|
||||
super().__init__()
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue