From 6c6e33e001d4a69544ac6ebd52885e54e8bacd13 Mon Sep 17 00:00:00 2001 From: Yuhong Sun Date: Mon, 22 Apr 2024 19:01:58 -0700 Subject: [PATCH] Allow Empty API Names (#77) --- backend/ee/danswer/db/api_key.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/backend/ee/danswer/db/api_key.py b/backend/ee/danswer/db/api_key.py index 68365a2b6..a8b1c0857 100644 --- a/backend/ee/danswer/db/api_key.py +++ b/backend/ee/danswer/db/api_key.py @@ -97,16 +97,15 @@ def update_api_key( if existing_api_key is None: raise ValueError(f"API key with id {api_key_id} does not exist") - new_name = api_key_args.name or UNNAMED_KEY_PLACEHOLDER - - existing_api_key.name = new_name + existing_api_key.name = api_key_args.name api_key_user = db_session.scalar( select(User).where(User.id == existing_api_key.user_id) # type: ignore ) if api_key_user is None: raise RuntimeError("API Key does not have associated user.") - api_key_user.email = get_api_key_fake_email(new_name, str(api_key_user.id)) + email_name = api_key_args.name or UNNAMED_KEY_PLACEHOLDER + api_key_user.email = get_api_key_fake_email(email_name, str(api_key_user.id)) db_session.commit() return ApiKeyDescriptor(