Feat: Features update #14
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…d detailed results
…with hashed passwords from backup
- Updated SlaveNode interface to include sync configuration and metrics. - Implemented database migration for slave nodes and sync logs. - Created API endpoints for registering, updating, deleting, and syncing slave nodes. - Added middleware for validating slave API keys. - Developed service layer for handling slave node operations. - Implemented query options for fetching slave nodes and their statuses. - Enhanced mock data for slave nodes to include new fields.
- Introduced SystemConfig interface in types for managing node configurations. - Created migration for system_configs table with necessary fields and enum for node modes. - Implemented system configuration controller with endpoints to get, update, connect, disconnect, and test connection to master node. - Added routes for system configuration management. - Developed service methods for API interactions related to system configuration. - Implemented query options for fetching system configuration with automatic refetching.
…nd ACLs for clarity; remove unused URL validation utility
…rvice checks, dependency updates, and health checks
… before generating Prisma client and running migrations; streamline logging
…rectly and enhance logging for Prisma client generation, migrations, and seeding
…te script; streamline backend and frontend build processes
…from Prisma schema
There was a problem hiding this comment.
Pull Request Overview
This PR adds comprehensive master-slave node synchronization functionality to the Nginx Love UI system, allowing distributed configuration management across multiple nginx nodes.
Key changes:
- Implements dual-mode operation (Master/Slave) for distributed configuration management
- Adds real-time node synchronization with hash-based change detection
- Creates comprehensive node management interfaces with health monitoring
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/update.sh | Complete deployment script with service management and health checks |
| apps/web/src/types/index.ts | Extended SlaveNode interface with new sync/metrics fields and added SystemConfig type |
| apps/web/src/services/system-config.service.ts | Service layer for system configuration and master-slave connection management |
| apps/web/src/services/slave.service.ts | Service layer for slave node operations and sync management |
| apps/web/src/queries/*.ts | React Query configuration for data fetching with appropriate cache intervals |
| apps/web/src/mocks/data.ts | Updated mock data to support new SlaveNode interface |
| apps/web/src/components/pages/SlaveNodes.tsx | Complete rewrite to dual-mode tabbed interface for master/slave operations |
| apps/api/src/utils/slave-status-checker.ts | Background service for monitoring slave node health status |
| apps/api/src/routes/*.ts | New API routes for slave management, system config, and node synchronization |
| apps/api/src/middleware/slaveAuth.ts | Authentication middleware for slave node API access |
| apps/api/src/controllers/*.ts | Controllers for slave operations, system config, and sync functionality |
| apps/api/prisma/schema.prisma | Database schema additions for slave nodes, system config, and sync logging |
| apps/api/prisma/migrations/*.sql | Database migrations for new tables and enums |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| # Seed database | ||
| log "Seeding database..." | ||
| cd "${BACKEND_DIR}" | ||
| pnpm ts-node prisma/seed.ts >> "$LOG_FILE" 2>&1 || warn "Failed to seed database (this is normal if data already exists)" |
There was a problem hiding this comment.
There's an extra space between 'pnpm' and 'ts-node' which should be removed for consistency.
Suggested change
| pnpm ts-node prisma/seed.ts >> "$LOG_FILE" 2>&1 || warn "Failed to seed database (this is normal if data already exists)" | |
| pnpm ts-node prisma/seed.ts >> "$LOG_FILE" 2>&1 || warn "Failed to seed database (this is normal if data already exists)" |
|
Please retry analysis of this Pull-Request directly on SonarQube Cloud |
vncloudsco
added a commit
that referenced
this pull request
Oct 8, 2025
* Feat: Update Features Backup & Restore (#12) * feat: Update Features Backup & Restore * Feat: Slave Mode (#13) * feat: Update Features Backup & Restore * Feat: Features update (#14) * feat: Update software * Features update version and update noti Change Password (#15) * feat: Update Features Backup & Restore * Fix frontend error (#16) * Refactor services to use centralized API module and token storage * Feat: Enhance Slave Mode UI with mode switch button and update node mode mutation * feat: Improve SSLDialog layout with enhanced text wrapping for certificate fields * refactor: replace js-cookie with localStorage for token management (#17) * feat: add syncInterval and lastSyncHash columns to system_configs table (#18) * feat: Update project goal description and remove security recommendation * feat: Update project goal description and remove security recommendation * About readme (#21) * feat: Update project goal description and remove security recommendation * Refactor be (#22) * Refactor code structure for improved readability and maintainability * style: limit max height of certificate, private key, and chain input fields (#23) --------- Co-authored-by: SangND <dacsang97@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.