-
Notifications
You must be signed in to change notification settings - Fork 7
V1 2 7 initial merge into dev branch for deployment testing #68
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
Open
9technologygroup
wants to merge
75
commits into
dev
Choose a base branch
from
v1-2-7
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
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
- Detect when FQDN starts with digit (IP address) - Generate 2 random letter prefix for database/service names - Use prefixed names for DB_NAME, DB_USER, INSTANCE_USER - Keep original FQDN for nginx configs and URLs - Prevents PostgreSQL errors with numeric database names Example: 192.168.1.50 -> xy192_168_1_50 for services
- Add root permission check at startup - Install sudo automatically if not present (for Debian systems) - Add run_as_user() helper function with better error handling - Provide fallback PostgreSQL setup using 'su' when sudo unavailable - Replace all sudo -u commands with the new helper function - Better error messages for missing users/sudo This ensures compatibility with minimal Debian systems that don't have sudo by default.
Frontend: - ESLint: 8.53.0 → 9.17.0 (fixes deprecated warnings) - React/React-DOM: 18.2.0 → 18.3.1 - Axios: 1.6.2 → 1.7.9 - Chart.js: 4.4.0 → 4.4.7 - Date-fns: 2.30.0 → 4.1.0 (major update) - Express: 4.18.2 → 4.21.2 - HTTP-proxy-middleware: 2.0.6 → 3.0.3 - Lucide-react: 0.294.0 → 0.468.0 - React-router-dom: 6.20.1 → 6.31.0 - Add ESLint v9 flat config (eslint.config.js) - Add required @eslint/js and globals dependencies Backend: - Prisma: 5.7.0 → 6.1.0 - Express: 4.18.2 → 4.21.2 - Dotenv: 16.3.1 → 16.4.7 - Express-rate-limit: 7.1.5 → 7.5.0 - Express-validator: 7.0.1 → 7.2.0 - Helmet: 7.1.0 → 8.0.0 - UUID: 9.0.1 → 11.0.3 - Winston: 3.11.0 → 3.17.0 - Nodemon: 3.0.2 → 3.1.9 Fixes npm warnings: - rimraf < v4 deprecated - inflight memory leaks - glob < v9 deprecated - @humanwhocodes packages deprecated - ESLint v8 no longer supported
- Add detect_package_manager() function that prefers 'apt' over 'apt-get' - Use PKG_MANAGER variables throughout script for consistency - Support both modern Debian 13+ (apt) and older systems (apt-get) - Automatically detect and use the best available package manager - Better error handling when no package manager is found Fixes compatibility with: - Debian 13 (Trixie) and newer - Ubuntu 22.04+ (prefers apt) - Older Debian/Ubuntu systems (fallback to apt-get) All package operations now use the detected package manager: - System updates and upgrades - Prerequisites installation - Node.js, PostgreSQL, nginx, certbot installation
Frontend package.json fixes: - react-router-dom: ^6.31.0 → ^6.30.1 (6.31.0 doesn't exist) - postcss: ^8.5.1 → ^8.5.6 (use latest stable version) Setup script fixes: - Replace deprecated --production flag with --omit=dev - Resolves npm warning: 'npm WARN config production Use --omit=dev instead' Fixes npm install errors: - 'No matching version found for react-router-dom@^6.31.0' - Deprecated npm configuration warnings
- Add first_name and last_name to select clause in setup-admin endpoint - Add first_name and last_name to select clause in signup endpoint - Ensures frontend receives the name fields after user creation - Fixes issue where first/last names don't populate in UI after setup The data was being saved to database correctly but not returned in the API response, causing frontend to not display the names properly.
Make the stack faster to start.
…d BACKEND_PORT is not found Fixed imports for react imports for useMemo, useEffect etc from hosts and HostDetail files
…ost creation page
Lint & format code, add rules, hooks, and workflow
Also resolves entire user object being return to client, including password_hash...⚠️
- Stops frontend trying to make calls that require auth before auth has occured - Stops frontend making calls that aren't necessary before auth has occured - Implements state machine to better handle auth phases
Fixes page reload on add host button click.
Post-lint fixes
More fixes
…takeplace upon build
npm install --only=optional So that Vite can install dependancies it needs for multi Arch builds
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.