fix(oauth): improve name validation logic for tool providers

This commit is contained in:
Harry 2025-07-11 21:44:45 +08:00
parent ace6e11a6f
commit 31e1261ae2
1 changed files with 2 additions and 2 deletions

View File

@ -164,7 +164,7 @@ class BuiltinToolManageService:
cache.delete()
# update name if provided
if name is not None and db_provider.name != name:
if name is not None and name not in {"", db_provider.name}:
# check if the name is already used
if (
session.query(BuiltinToolProvider)
@ -221,7 +221,7 @@ class BuiltinToolManageService:
provider_controller.validate_credentials(user_id, credentials)
# generate name if not provided
if name is None:
if name is None or name == "":
name = BuiltinToolManageService.generate_builtin_tool_provider_name(
session=session, tenant_id=tenant_id, provider=provider, credential_type=api_type
)