-
Notifications
You must be signed in to change notification settings - Fork 26
fix cve issue and revamp clickdeploy to enable persistent status #72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR fixes CVE security issues and revamps the click deployment system to enable persistent status tracking. The changes implement comprehensive WebSocket-based monitoring and database persistence for deployment states.
- Updates form-data and setuptools versions to address security vulnerabilities
- Implements persistent deployment status tracking with database storage
- Revamps WebSocket monitoring system for real-time deployment status updates
Reviewed Changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| studio-frontend/packages/ui/src/ui-component/table/FlowListTable.jsx | Adds deployment status tracking, WebSocket monitoring, and status persistence |
| studio-frontend/packages/ui/src/ui-component/dialog/OneClickDeploymentDialog.jsx | Simplifies dialog by removing local WebSocket management and relying on parent component |
| studio-frontend/packages/ui/src/api/chatflows.js | Adds deployment status API endpoints |
| studio-frontend/packages/server/src/services/chatflows/index.ts | Implements deployment status service functions |
| studio-frontend/packages/server/src/routes/chatflows/index.ts | Adds deployment status routes |
| studio-frontend/packages/server/src/controllers/chatflows/index.ts | Implements deployment status controllers |
| studio-frontend/packages/server/src/database/entities/ChatFlow.ts | Adds deployment status fields to ChatFlow entity |
| studio-frontend/packages/server/src/database/migrations/*/1754700956637-AddDeploymentStatusToChatFlow.ts | Database migrations for deployment status fields |
| studio-frontend/package.json | Updates form-data version for CVE fix |
| studio-backend/app/services/clickdeploy_service.py | Completely revamps deployment service with async processing and status monitoring |
| studio-backend/app/routers/clickdeploy_router.py | Simplifies router to use new async deployment service |
| studio-backend/Dockerfile | Updates setuptools version for security |
| app-backend/Dockerfile | Updates setuptools and adds additional package versions |
Comments suppressed due to low confidence (1)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Signed-off-by: wwanarif <wan.abdul.hakim.b.wan.arif@intel.com>
Signed-off-by: wwanarif <wan.abdul.hakim.b.wan.arif@intel.com>
siawchen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
No description provided.