forked from phoenix/litellm-mirror
add enforcement for unique key aliases
This commit is contained in:
parent
21156ff5d0
commit
74c025c50c
1 changed files with 37 additions and 0 deletions
|
@ -420,6 +420,11 @@ async def generate_key_fn( # noqa: PLR0915
|
||||||
|
|
||||||
data_json.pop("tags")
|
data_json.pop("tags")
|
||||||
|
|
||||||
|
await _enforce_unique_key_alias(
|
||||||
|
key_alias=data_json.get("key_alias", None),
|
||||||
|
prisma_client=prisma_client,
|
||||||
|
)
|
||||||
|
|
||||||
response = await generate_key_helper_fn(
|
response = await generate_key_helper_fn(
|
||||||
request_type="key", **data_json, table_name="key"
|
request_type="key", **data_json, table_name="key"
|
||||||
)
|
)
|
||||||
|
@ -585,6 +590,11 @@ async def update_key_fn(
|
||||||
data=data, existing_key_row=existing_key_row
|
data=data, existing_key_row=existing_key_row
|
||||||
)
|
)
|
||||||
|
|
||||||
|
await _enforce_unique_key_alias(
|
||||||
|
key_alias=non_default_values.get("key_alias", None),
|
||||||
|
prisma_client=prisma_client,
|
||||||
|
)
|
||||||
|
|
||||||
response = await prisma_client.update_data(
|
response = await prisma_client.update_data(
|
||||||
token=key, data={**non_default_values, "token": key}
|
token=key, data={**non_default_values, "token": key}
|
||||||
)
|
)
|
||||||
|
@ -1883,3 +1893,30 @@ async def test_key_logging(
|
||||||
status="healthy",
|
status="healthy",
|
||||||
details=f"No logger exceptions triggered, system is healthy. Manually check if logs were sent to {logging_callbacks} ",
|
details=f"No logger exceptions triggered, system is healthy. Manually check if logs were sent to {logging_callbacks} ",
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
async def _enforce_unique_key_alias(
|
||||||
|
key_alias: Optional[str],
|
||||||
|
prisma_client: Any,
|
||||||
|
) -> None:
|
||||||
|
"""
|
||||||
|
Helper to enforce unique key aliases across all keys.
|
||||||
|
|
||||||
|
Args:
|
||||||
|
key_alias (Optional[str]): The key alias to check
|
||||||
|
prisma_client (Any): Prisma client instance
|
||||||
|
|
||||||
|
Raises:
|
||||||
|
HTTPException: If key alias already exists
|
||||||
|
"""
|
||||||
|
if key_alias is not None and prisma_client is not None:
|
||||||
|
existing_key = await prisma_client.db.litellm_verificationtoken.find_first(
|
||||||
|
where={"key_alias": key_alias}
|
||||||
|
)
|
||||||
|
if existing_key is not None:
|
||||||
|
raise HTTPException(
|
||||||
|
status_code=400,
|
||||||
|
detail={
|
||||||
|
"error": f"Key alias '{key_alias}' already exists. Unique key aliases across all keys are required."
|
||||||
|
},
|
||||||
|
)
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue