1047 Commits

Author SHA1 Message Date
courtmanr@gmail.com 4b53f8d47f chore: remove docs directory with large PDF files
- Removes 22MB of Proxmox documentation PDFs
- Reduces release tarball size from 25MB to ~11MB
- Documentation can be accessed directly from Proxmox sources
v3.20.0
2025-06-02 22:00:37 +01:00
courtmanr@gmail.com 4c67aab7e8 chore: release v3.20.0 2025-06-02 18:50:19 +01:00
courtmanr@gmail.com 513e31126e feat: redesign backup health card with clearer statistics and improved layout
- Make health score more prominent with larger font and context
- Replace confusing guest counts with clear total backup count
- Add recent coverage metric showing guests with backups in last 7 days
- Reorganize backup type statistics with clear labeling
- Add coverage ratio showing protected vs unprotected guests
- Improve visual hierarchy and information density

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 18:48:08 +01:00
courtmanr@gmail.com 066e61e95e feat: improve backup type labels with styled badges in history card
- Convert plain text backup type labels to rounded badge components
- Add proper spacing between multiple backup types (PBS, PVE, SNAP)
- Implement consistent styling across multi-date and single-date views
- Include dark mode support with appropriate color contrasts
- Left-align types column for better visual consistency

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 18:20:06 +01:00
courtmanr@gmail.com 9b74f2b0b4 test: add timestamp validation regression test
- Create comprehensive test for getValidTimestamp() method
- Verify handling of valid, invalid, null, and undefined timestamps
- Confirm no RangeError exceptions are thrown during ISO string generation
- Validate the complete fix for Teams webhook "Time Value Error"
- Demonstrate robust fallback logic for all edge cases

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 18:08:30 +01:00
courtmanr@gmail.com f4aaea1f53 fix: enhance webhook timestamp validation and add comprehensive tests
- Add robust timestamp validation with getValidTimestamp() method
- Handle invalid timestamp values gracefully with fallback to current time
- Prevent RangeError when Date constructor receives invalid values
- Add comprehensive AlertManager test suite with 12 test cases
- Test webhook payload structure, timestamp handling, and error scenarios
- Verify Discord/Slack compatibility and email notification fixes
- Increase AlertManager test coverage from 0% to 40.11%

Fixes #114 completely with bulletproof timestamp handling

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 18:05:26 +01:00
courtmanr@gmail.com 78d6f7ff15 fix: resolve remaining Teams webhook timestamp errors
- Fix remaining alert.timestamp references in email notifications and Discord/Slack embeds
- Replace with alert.triggeredAt || alert.lastUpdate || Date.now() pattern
- Resolves RangeError when sending webhook notifications to Teams
- Fixes issue #114 completely

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 18:01:21 +01:00
courtmanr@gmail.com 5d16441e1e test: add webhook functionality testing tools
- Add test-webhook.js server for receiving webhook notifications
- Add webhook-integration-test.js for comprehensive webhook testing
- Verify webhook payload format and Discord/Slack compatibility
- Test alert system integration with webhook notifications

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 17:58:35 +01:00
courtmanr@gmail.com 3775b4d68a fix: resolve webhook notification timestamp error and improve scroll timing
- Fix RangeError in AlertManager by using correct timestamp fields (triggeredAt/lastUpdate) instead of non-existent alert.timestamp
- Replace Promise.resolve() with requestAnimationFrame() for better scroll position restoration timing

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 16:53:49 +01:00
courtmanr@gmail.com 2eec1c32e4 feat: implement consistent backup type filtering across detail card views
- Add comprehensive backup type filtering to detail card when filters are active
- Show only filtered backup types (PBS/PVE/SNAP) instead of all types
- Ensure calendar date selections respect active backup type filters
- Implement three helper functions for different data structures
- Clean up debug logging and improve scroll position handling

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 16:51:00 +01:00
courtmanr@gmail.com cbd6964ac8 fix: improve backup health monitoring and add comprehensive validation
Enhanced backup health status calculations with more accurate age-based
categorization and improved UI filtering. Added comprehensive backup data
validation test suite and ground truth testing framework.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 15:20:59 +01:00
courtmanr@gmail.com 8ff6f8f366 test: significantly improve test coverage and fix Jest configuration
## Test Coverage Achievements
- **apiClients.js**: 100% statement coverage 
- **configLoader.js**: 100% statement coverage 
- **pbsUtils.js**: 100% statement coverage 
- **Overall coverage**: 77.97% (up from 77.83%)
- **All tests passing**: 116/116 

## Key Improvements

### Jest Configuration
- Fix Jest configuration to use V8 coverage provider instead of Babel
- Resolve Babel/Istanbul errors that were preventing tests from running
- Add `testEnvironment: "node"` and `coverageProvider: "v8"` settings

### Test Coverage Enhancements
- **pbsUtils.js**: Export and test `categorizeAndCountTasks` function for complete coverage
- **configLoader.js**: Add test for config directory loading path
- **dataFetcher.js**: Add defensive null checking for API clients and comprehensive test fixes
- **All test files**: Update test expectations to match current application behavior

### Test Reliability
- Fix config tests to match new setup mode behavior instead of error throwing
- Update dataFetcher tests for correct API call sequences and error messages
- Add proper mock data for backup functionality testing
- Resolve test flakiness with proper async handling

### Code Quality
- Add null safety check in dataFetcher for missing API client endpoints
- Export previously internal function for better testability
- Maintain backward compatibility while improving robustness

## Files Changed
- `package.json`: Update Jest configuration
- `server/pbsUtils.js`: Export categorizeAndCountTasks function
- `server/dataFetcher.js`: Add null client validation
- `server/tests/*.test.js`: Comprehensive test updates and additions

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 09:49:48 +01:00
courtmanr@gmail.com 945c3e4320 fix: upgrade Nodemailer to 7.0.3 and resolve Express 5 route conflicts
- Upgraded Nodemailer 6.10.1 → 7.0.3 (fully compatible with SMTP usage)
- Temporarily keep Express 4.21.2 due to path-to-regexp v8 breaking changes
- Removed duplicate threshold routes from configApi.js (moved to thresholdRoutes.js)
- Express 5 upgrade will require route pattern updates for compatibility

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 09:47:21 +01:00
courtmanr@gmail.com ade598d295 chore: upgrade Express to 5.1.0 and Nodemailer to 7.0.3
- Express 4.21.2 → 5.1.0: Major version upgrade with improved performance
- Nodemailer 6.10.1 → 7.0.3: Major version upgrade with SESv2 support
- Both upgrades are fully compatible with existing codebase
- No breaking changes affect current SMTP usage or Express routes
- Automated dependency security and feature improvements

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-02 09:38:41 +01:00
courtmanr@gmail.com 0442ca832b fix: resolve auto-update npm permission errors and dependency installation issues
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-01 23:37:05 +01:00
courtmanr@gmail.com 6b9c464072 chore: release v3.19.0 v3.19.0 2025-06-01 23:27:39 +01:00
courtmanr@gmail.com 29e90d769d fix: resolve PVE backup detection and cross-cluster guest ID collisions
- Fix PVE backup task classification logic that incorrectly defaulted to PBS when log parsing failed
- Implement proper separation between PVE backups (actual backup files) and snapshots (point-in-time states)
- Add cross-node backup matching within clusters while maintaining cross-cluster isolation
- Resolve duplicate guest display issue caused by identical guest IDs across different Proxmox clusters
- Remove reliance on unparseable job-level backup tasks in favor of actual backup file detection

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-01 23:25:11 +01:00
courtmanr@gmail.com ab88beee2e feat: add dynamic alert rule management with cleanup for disabled types
- Add refreshRules() method to AlertManager for dynamic rule updates
- Add cleanupAlertsForRule() to remove active alerts when rules are disabled
- Modify configuration reload to trigger alert rule refresh
- Add getAlertManager() method to state module for consistent access
- Existing alerts are now properly cleaned up when global alert types are disabled
- Changes take effect immediately without requiring server restart

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-01 22:35:30 +01:00
courtmanr@gmail.com 7ac2eeb869 debug: add logging to auto-update mechanism to troubleshoot download URL issue 2025-06-01 21:51:53 +01:00
courtmanr@gmail.com eb31436868 fix: use existing node user (uid:gid 1000:1000) instead of creating new user
- Leverages node:18-alpine's existing node user with proper uid:gid 1000:1000
- Eliminates uid/gid conflicts during Docker build
- Maintains security by avoiding system service account permissions
- Completes fix for issue #109
v3.18.0
2025-06-01 21:42:49 +01:00
courtmanr@gmail.com 9460594667 fix: change appuser to uid:gid 1000:1000 for security
- Use standard user uid:gid 1000:1000 instead of 100:101 (postfix:crontab)
- Add user override in docker-compose.yml to ensure consistent uid:gid
- Prevents .env files from having system service account permissions
- Addresses critical security concern in issue #109
2025-06-01 21:40:04 +01:00
courtmanr@gmail.com cf4ea9dad1 fix: ensure config and data directories exist with proper permissions in Docker container
- Create /usr/src/app/config and /usr/src/app/data directories before setting ownership
- Fixes permission issues when saving configuration via web UI
- Resolves issue #109 where appuser cannot write to .env file in mounted config volume
2025-06-01 21:35:46 +01:00
courtmanr@gmail.com 3ba287fd50 chore: release v3.18.0 2025-06-01 21:30:12 +01:00
courtmanr@gmail.com 7c21d73844 feat: implement webhook notifications for alerts
- Add comprehensive webhook support for Discord, Slack, Teams
- Rich embeds with color-coded severity and inline fields
- Webhook configuration UI with test functionality
- Dual payload format (Discord embeds + Slack attachments)
- Error handling with timeout and proper HTTP responses
- Test webhook endpoint with sample alert data

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-01 21:16:47 +01:00
courtmanr@gmail.com cacad13af9 fix: correct nodemailer method name from createTransporter to createTransport
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-01 21:08:16 +01:00
courtmanr@gmail.com 29889e5e10 feat: implement email notifications for alerts
- Add nodemailer dependency for SMTP email sending
- Implement email notification channel in AlertManager
- Add rich HTML email templates with alert details
- Create email configuration UI in settings page
- Add test email functionality with validation
- Support for multiple recipients and Gmail app passwords
- Addresses feature request in issue #111

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-06-01 21:03:28 +01:00
courtmanr@gmail.com 1d3fb22e48 fix: resolve custom threshold dropdown validation error
- Client now uses 'auto-detect' placeholder when node can't be determined
- Server handles node auto-detection by looking up VM/LXC in current state
- Fixes 'Please select a VM/LXC from the dropdown first' error in v3.17.3
- Addresses issue #110
2025-06-01 20:51:53 +01:00
courtmanr@gmail.com 9523589f3c fix: enhance security by properly sanitizing sensitive data in diagnostic reports
- Add sanitization for name fields in Proxmox and PBS configurations
- Sanitize PBS node_name values to prevent leaking actual node names
- Improve URL sanitization to handle URLs without protocols
- Use clearer redaction patterns (REDACTED-IP, REDACTED-HOST)
- Add more aggressive fallback sanitization for malformed URLs

This prevents exposure of sensitive infrastructure details when diagnostic
reports are shared for debugging purposes.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 23:57:48 +01:00
courtmanr@gmail.com bd88efc443 feat: make node names clickable in summary cards
Added clickable links to node names in both regular and mobile summary cards.
When a node URL is available, users can now click the node name to open the
node's web interface in a new tab.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 23:50:49 +01:00
courtmanr@gmail.com b58baa9d14 fix: resolve update mechanism issues and add test mode
- Fix download URL property mismatch (downloadUrl vs browser_download_url)
- Add comprehensive error logging for update failures
- Create test mode for update mechanism (UPDATE_TEST_MODE=true)
- Add test script for easy update testing without releases
- Improve restart logic for different deployment environments
- Add tar package dependency for creating update packages
- Update .gitignore to exclude backup and temp directories
- Add documentation for update testing workflow

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 23:38:08 +01:00
courtmanr@gmail.com d0c7b9fbc1 3.17.3 v3.17.3 2025-05-31 22:57:26 +01:00
courtmanr@gmail.com 8d838bb26e fix: prioritize display names in node link resolution
When custom node names are configured, the dashboard shows display names
but node links were failing because getHostUrl was checking actual node
names first. This fix reorders the lookup to check display names first,
ensuring that custom-named nodes have working links.

- Check display names before actual node names in getHostUrl
- Maintains backward compatibility for non-custom setups
- Fixes issue where clicking custom node names didn't open web interface

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 22:56:56 +01:00
courtmanr@gmail.com ff17a1786a 3.17.2 v3.17.2 2025-05-31 22:36:12 +01:00
courtmanr@gmail.com 65a556bc68 fix: update Express to 4.21.2 to resolve security vulnerabilities
Fixes 7 Dependabot security alerts:
- HIGH: path-to-regexp ReDoS vulnerabilities (CVE-2024-45296, CVE-2024-45813)
- HIGH: body-parser DoS vulnerability (CVE-2024-45590)
- LOW: cookie out of bounds characters (CVE-2024-47764)
- LOW: express XSS via response.redirect() (CVE-2024-43796)
- LOW: send/serve-static template injection XSS (CVE-2024-43799, CVE-2024-43800)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 22:33:31 +01:00
courtmanr@gmail.com 0cfce98e16 chore: release v3.17.1 v3.17.1 2025-05-31 22:10:02 +01:00
courtmanr@gmail.com 2b2b2680e8 fix: resolve stuck tooltip issue with improved event handling
- Fixed handleMouseOut to properly detect when leaving tooltip triggers
- Added document mouseleave handler to hide tooltips when mouse leaves viewport
- Added scroll event handler to prevent tooltips getting stuck during scrolling
- Improved event handling to prevent premature tooltip hiding from child element bubbling

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 22:04:40 +01:00
courtmanr@gmail.com 938947bd00 fix: only use custom node names when explicitly configured
- Change config loader to set name to null instead of host address when no custom name is provided
- This ensures discovered node names are used by default
- Custom names are only applied when PROXMOX_NODE_NAME is explicitly set

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 21:51:32 +01:00
courtmanr@gmail.com 8aa47a5261 fix: improve node name display and configuration handling
- Add support for custom display names via PROXMOX_NODE_NAME configuration
- Handle multi-node clusters properly by prefixing node names with endpoint name
- Fix dashboard and nodes views to show configured display names
- Fix non-sequential endpoint numbering issue in settings (node_2, node_8, node_14)
- Improve endpoint configuration loading to handle any index number

Fixes #100

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 21:41:01 +01:00
courtmanr@gmail.com 2080d2ccaa fix: remove diagnostics modal icon from UI
- Removed diagnostics icon button from header
- Removed JavaScript code that managed icon visibility
- Removed event listener for diagnostics icon click
- Diagnostics functionality remains available in settings menu

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 21:16:19 +01:00
courtmanr@gmail.com 38aee506db fix: preserve .env file during installer updates
- Backup .env file before removing old installation
- Restore .env file after extracting new version
- Fix both update and migration processes
- Ensure proper ownership and permissions on restored .env file
2025-05-31 19:55:50 +01:00
courtmanr@gmail.com 3bf6d64a80 fix: implement Apply Update functionality in settings
- Store release data when checking for updates
- Use server API for update checks to get Docker status
- Implement proper applyUpdate function with progress tracking
- Handle Docker deployments with appropriate warning message
- Add WebSocket listeners for update progress events
2025-05-31 18:25:51 +01:00
courtmanr@gmail.com 12bf7d2473 chore: release v3.17.0 v3.17.0 2025-05-31 18:12:32 +01:00
courtmanr@gmail.com 93f9f98e3c fix: prevent PBS host from saving with protocol prefix
- Strip http:// or https:// from PBS_HOST values in settings form
- Update placeholders to show IP/hostname format without protocol
- Fix discovery cycle to trigger for PBS-only configurations
- Resolves issue where PBS data wouldn't load due to malformed host URL

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 18:04:10 +01:00
courtmanr@gmail.com 012825b93b feat: integrate diagnostics into settings menu
- Move diagnostics from standalone page to settings modal tab
- Add Diagnostics tab to settings navigation
- Port all diagnostic functionality to settings UI module
- Update diagnostics button to open settings modal directly
- Remove standalone diagnostics.html page
- Preserve all features: report generation, sanitization, copy/download

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 17:46:19 +01:00
courtmanr@gmail.com 81c07e118c fix: support non-sequential server IDs in configuration
Previously, the config loader required sequential numbering (2, 3, 4...) for additional Proxmox/PBS endpoints. If a user had PROXMOX_HOST_2 and PROXMOX_HOST_4 (skipping 3), only endpoint 2 would be loaded.

This fix scans all environment variables to find any PROXMOX_HOST_N or PBS_HOST_N patterns, regardless of numbering sequence. Now users can have endpoints numbered 2, 5, 10, etc. and all will be properly loaded.

Fixes #96

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 17:30:04 +01:00
courtmanr@gmail.com 7fb1a14cff chore: update installer SHA to latest commit 2025-05-31 15:56:43 +01:00
courtmanr@gmail.com 34ea6d9d9e fix: use full SHA for installer version tracking
- Use full 40-char SHA instead of abbreviated 7-char version
- Fixes issue where installer kept detecting updates
- SHA comparison now works correctly
2025-05-31 15:56:23 +01:00
courtmanr@gmail.com 90628a5ef2 chore: update installer SHA to current commit 2025-05-31 15:53:18 +01:00
courtmanr@gmail.com 6798dd54db feat: restore robust installer self-update using GitHub API
- Added SHA-based version tracking to avoid GitHub CDN caching issues
- Uses GitHub API to check latest commit SHA for the installer
- Falls back to simple diff check if jq is not available
- Updates embedded SHA when downloading new version
- Much more reliable than simple file comparison

This restores the update method from the original installer that
properly handles GitHub's aggressive CDN caching.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 15:52:59 +01:00
courtmanr@gmail.com 6c2236a9a7 Revert "chore: bump version to 3.16.2"
This reverts commit c466a58f7a.
2025-05-31 15:47:55 +01:00