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

address greptile review feedback (greploop iteration 13)

Browse source
This commit is contained in:
Ishaan Jaffer 2026-03-05 16:15:11 -08:00
parent dfac66ed51
commit cbb6b6700f
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
litellm/proxy/management_endpoints/ui_sso.py
View file

@ -2616,9 +2616,9 @@ class SSOAuthenticationHandler:
post_kwargs: Dict[str, Any] = {
"data": token_data,
"headers": {
"Content-Type": "application/x-www-form-urlencoded",
"Accept": "application/json",
**additional_headers,
"Content-Type": "application/x-www-form-urlencoded", # must not be overridden
"Accept": "application/json",
},
"timeout": 30.0,
}
@ -2719,8 +2719,8 @@ class SSOAuthenticationHandler:
resp = await client.get(
userinfo_endpoint,
headers={
"Authorization": f"Bearer {access_token}",
**additional_headers,
"Authorization": f"Bearer {access_token}", # must not be overridden
},
timeout=30.0,
)

2
tests/test_litellm/proxy/management_endpoints/test_ui_sso.py
View file

@ -4529,6 +4529,8 @@ async def test_pkce_token_exchange_basic_auth():
# Verify Basic Auth is set
assert "auth" in kwargs
assert isinstance(kwargs["auth"], httpx.BasicAuth)
# Verify code_verifier is in the POST body (essential PKCE field)
assert kwargs.get("data", {}).get("code_verifier") == "verifier_abc"
return mock_response
with patch("litellm.proxy.management_endpoints.ui_sso.httpx.AsyncClient") as mock_client_cls: