Skip to content

✨ Error Handler Framework#2596

Merged
WMC001 merged 2 commits intodevelopfrom
develop_error_framework_v2
Mar 4, 2026
Merged

✨ Error Handler Framework#2596
WMC001 merged 2 commits intodevelopfrom
develop_error_framework_v2

Conversation

@Zhi-a
Copy link
Contributor

@Zhi-a Zhi-a commented Mar 2, 2026

✨ Add FastAPI application factory and unit tests: Implemented a new app_factory module for creating FastAPI applications with common configurations and exception handling. Added comprehensive unit tests to validate the functionality of the create_app and register_exception_handlers functions.

未适配新错误码框架的模块,采用以前的方式抛出错误。
已适配模块(dify),采用新框架抛错

@Zhi-a Zhi-a requested review from Phinease and WMC001 as code owners March 2, 2026 07:12
@Zhi-a Zhi-a force-pushed the develop_error_framework_v2 branch 5 times, most recently from 20ca44a to 58a573b Compare March 2, 2026 12:26
@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

❌ Patch coverage is 97.83550% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
backend/apps/app_factory.py 93.75% 1 Missing and 1 partial ⚠️
backend/middleware/exception_handler.py 94.59% 1 Missing and 1 partial ⚠️
backend/services/dify_service.py 95.23% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Zhi-a Zhi-a changed the title [WIP]Develop error framework v2 Develop error framework v2 Mar 3, 2026
@Zhi-a Zhi-a changed the title Develop error framework v2 ✨ Error Handler Framework Mar 3, 2026
…pp_factory module for creating FastAPI applications with common configurations and exception handling. Added comprehensive unit tests to validate the functionality of the create_app and register_exception_handlers functions.
@Zhi-a Zhi-a force-pushed the develop_error_framework_v2 branch from 2029e29 to a3c89df Compare March 4, 2026 02:22
@WMC001 WMC001 merged commit 448caa4 into develop Mar 4, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants