QT/superset2
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2026-05-12 19:35:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(mcp): classify user errors as WARNING, system errors as ERROR (#39634)

Browse Source 
Co-authored-by: Elizabeth Thompson <eschutho@gmail.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Amin Ghadersohi
2026-04-28 08:55:17 -04:00
committed by GitHub
parent 7bee2afa8e
commit 6947881ba7
13 changed files with 458 additions and 39 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
superset/mcp_service/sql_lab/tool/execute_sql.py
View File

@@ -95,7 +95,7 @@ async def execute_sql(request: ExecuteSqlRequest, ctx: Context) -> ExecuteSqlRes
db.session.query(Database).filter_by(id=request.database_id).first()
)
if not database:
await ctx.error(
await ctx.warning(
"Database not found: database_id=%s" % request.database_id
)
return ExecuteSqlResponse(
@@ -105,7 +105,7 @@ async def execute_sql(request: ExecuteSqlRequest, ctx: Context) -> ExecuteSqlRes
)
if not security_manager.can_access_database(database):
await ctx.error(
await ctx.warning(
"Access denied to database: %s" % database.database_name
)
return ExecuteSqlResponse(
@@ -153,7 +153,7 @@ async def execute_sql(request: ExecuteSqlRequest, ctx: Context) -> ExecuteSqlRes
return response
except OAuth2RedirectError as ex:
await ctx.error(
await ctx.warning(
"Database requires OAuth authentication: database_id=%s"
% request.database_id
)