phoenix/litellm
1
0
Fork 0
forked from phoenix/litellm-mirror
Code Pull requests Activity

test(test_key_generate_prisma.py): fix test to only let admin delete a key

Browse source
This commit is contained in:
Krrish Dholakia 2024-03-11 14:18:01 -07:00
parent 2addd66393
commit d1644db8ce
2 changed files with 15 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
litellm/proxy/proxy_server.py
View file

@ -3776,6 +3776,9 @@ async def delete_key_fn(
## only allow user to delete keys they own ## only allow user to delete keys they own
user_id = user_api_key_dict.user_id user_id = user_api_key_dict.user_id
verbose_proxy_logger.debug(
f"user_api_key_dict.user_role: {user_api_key_dict.user_role}"
)
if ( if (
user_api_key_dict.user_role is not None user_api_key_dict.user_role is not None
and user_api_key_dict.user_role == "proxy_admin" and user_api_key_dict.user_role == "proxy_admin"

13
litellm/tests/test_key_generate_prisma.py
View file

@ -737,8 +737,19 @@ def test_delete_key(prisma_client):
delete_key_request = KeyRequest(keys=[generated_key]) delete_key_request = KeyRequest(keys=[generated_key])
bearer_token = "Bearer sk-1234"
request = Request(scope={"type": "http"})
request._url = URL(url="/key/delete")
# use generated key to auth in
result = await user_api_key_auth(request=request, api_key=bearer_token)
print(f"result: {result}")
result.user_role = "proxy_admin"
# delete the key # delete the key
result_delete_key = await delete_key_fn(data=delete_key_request) result_delete_key = await delete_key_fn(
data=delete_key_request, user_api_key_dict=result
)
print("result from delete key", result_delete_key) print("result from delete key", result_delete_key)
assert result_delete_key == {"deleted_keys": [generated_key]} assert result_delete_key == {"deleted_keys": [generated_key]}