Skip to content

Commit

Permalink
Merge pull request #3750 from BerriAI/litellm_raise_exception_when_de…
Browse files Browse the repository at this point in the history
…leting_updating_team

[Fix] - raise Exception when trying to update/delete a non-existent team
  • Loading branch information
ishaan-jaff authored May 21, 2024
2 parents 1006284 + 367d860 commit 829ca84
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions litellm/proxy/proxy_server.py
Original file line number Diff line number Diff line change
Expand Up @@ -7522,6 +7522,11 @@ async def update_team(
existing_team_row = await prisma_client.get_data(
team_id=data.team_id, table_name="team", query_type="find_unique"
)
if existing_team_row is None:
raise HTTPException(
status_code=404,
detail={"error": f"Team not found, passed team_id={data.team_id}"},
)

updated_kv = data.json(exclude_none=True)
team_row = await prisma_client.update_data(
Expand Down Expand Up @@ -7775,6 +7780,17 @@ async def delete_team(
if data.team_ids is None:
raise HTTPException(status_code=400, detail={"error": "No team id passed in"})

# check that all teams passed exist
for team_id in data.team_ids:
team_row = await prisma_client.get_data( # type: ignore
team_id=team_id, table_name="team", query_type="find_unique"
)
if team_row is None:
raise HTTPException(
status_code=404,
detail={"error": f"Team not found, passed team_id={team_id}"},
)

## DELETE ASSOCIATED KEYS
await prisma_client.delete_data(team_id_list=data.team_ids, table_name="key")
## DELETE TEAMS
Expand Down

0 comments on commit 829ca84

Please sign in to comment.