mirror of
https://git.mirrors.martin98.com/https://github.com/langgenius/dify.git
synced 2025-08-14 01:15:56 +08:00
fix(api/ops_trace): avoid raise exception directly (#11732)
Signed-off-by: -LAN- <laipz8200@outlook.com>
This commit is contained in:
parent
7b7eb00385
commit
259cff9f22
@ -65,7 +65,7 @@ class ModelConfigResource(Resource):
|
|||||||
provider_type=agent_tool_entity.provider_type,
|
provider_type=agent_tool_entity.provider_type,
|
||||||
identity_id=f"AGENT.{app_model.id}",
|
identity_id=f"AGENT.{app_model.id}",
|
||||||
)
|
)
|
||||||
except Exception as e:
|
except Exception:
|
||||||
continue
|
continue
|
||||||
|
|
||||||
# get decrypted parameters
|
# get decrypted parameters
|
||||||
@ -97,7 +97,7 @@ class ModelConfigResource(Resource):
|
|||||||
app_id=app_model.id,
|
app_id=app_model.id,
|
||||||
agent_tool=agent_tool_entity,
|
agent_tool=agent_tool_entity,
|
||||||
)
|
)
|
||||||
except Exception as e:
|
except Exception:
|
||||||
continue
|
continue
|
||||||
|
|
||||||
manager = ToolParameterConfigurationManager(
|
manager = ToolParameterConfigurationManager(
|
||||||
|
@ -1,4 +1,5 @@
|
|||||||
from flask_restful import Resource, reqparse
|
from flask_restful import Resource, reqparse
|
||||||
|
from werkzeug.exceptions import BadRequest
|
||||||
|
|
||||||
from controllers.console import api
|
from controllers.console import api
|
||||||
from controllers.console.app.error import TracingConfigCheckError, TracingConfigIsExist, TracingConfigNotExist
|
from controllers.console.app.error import TracingConfigCheckError, TracingConfigIsExist, TracingConfigNotExist
|
||||||
@ -26,7 +27,7 @@ class TraceAppConfigApi(Resource):
|
|||||||
return {"has_not_configured": True}
|
return {"has_not_configured": True}
|
||||||
return trace_config
|
return trace_config
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
raise e
|
raise BadRequest(str(e))
|
||||||
|
|
||||||
@setup_required
|
@setup_required
|
||||||
@login_required
|
@login_required
|
||||||
@ -48,7 +49,7 @@ class TraceAppConfigApi(Resource):
|
|||||||
raise TracingConfigCheckError()
|
raise TracingConfigCheckError()
|
||||||
return result
|
return result
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
raise e
|
raise BadRequest(str(e))
|
||||||
|
|
||||||
@setup_required
|
@setup_required
|
||||||
@login_required
|
@login_required
|
||||||
@ -68,7 +69,7 @@ class TraceAppConfigApi(Resource):
|
|||||||
raise TracingConfigNotExist()
|
raise TracingConfigNotExist()
|
||||||
return {"result": "success"}
|
return {"result": "success"}
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
raise e
|
raise BadRequest(str(e))
|
||||||
|
|
||||||
@setup_required
|
@setup_required
|
||||||
@login_required
|
@login_required
|
||||||
@ -85,7 +86,7 @@ class TraceAppConfigApi(Resource):
|
|||||||
raise TracingConfigNotExist()
|
raise TracingConfigNotExist()
|
||||||
return {"result": "success"}
|
return {"result": "success"}
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
raise e
|
raise BadRequest(str(e))
|
||||||
|
|
||||||
|
|
||||||
api.add_resource(TraceAppConfigApi, "/apps/<uuid:app_id>/trace-config")
|
api.add_resource(TraceAppConfigApi, "/apps/<uuid:app_id>/trace-config")
|
||||||
|
@ -948,7 +948,7 @@ class DocumentRetryApi(DocumentResource):
|
|||||||
if document.indexing_status == "completed":
|
if document.indexing_status == "completed":
|
||||||
raise DocumentAlreadyFinishedError()
|
raise DocumentAlreadyFinishedError()
|
||||||
retry_documents.append(document)
|
retry_documents.append(document)
|
||||||
except Exception as e:
|
except Exception:
|
||||||
logging.exception(f"Failed to retry document, document id: {document_id}")
|
logging.exception(f"Failed to retry document, document id: {document_id}")
|
||||||
continue
|
continue
|
||||||
# retry document
|
# retry document
|
||||||
|
Loading…
x
Reference in New Issue
Block a user