From 31e1261ae22b4715aa12735b3a8a00882cb0869a Mon Sep 17 00:00:00 2001 From: Harry Date: Fri, 11 Jul 2025 21:44:45 +0800 Subject: [PATCH] fix(oauth): improve name validation logic for tool providers --- api/services/tools/builtin_tools_manage_service.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/api/services/tools/builtin_tools_manage_service.py b/api/services/tools/builtin_tools_manage_service.py index 4229e3bb9c..6fda9591ce 100644 --- a/api/services/tools/builtin_tools_manage_service.py +++ b/api/services/tools/builtin_tools_manage_service.py @@ -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 )