[PR #443] [MERGED] Improve server startup logging and update script fetching #447

New issue

Closed

opened 2026-02-26 12:41:24 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 12:41:24 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE-Local/pull/443
Author: @MickLesk
Created: 1/13/2026
Status: ✅ Merged
Merged: 1/13/2026
Merged by: @MickLesk

Base: main ← Head: get_update_script

📝 Commits (1)

60f81fa Improve server startup logging and update script fetching

📊 Changes

2 files changed (+27 additions, -0 deletions)

View changed files

📝 server.js (+6 -0)
📝 src/server/api/routers/version.ts (+21 -0)

📄 Description

✍️ Description

Adds success and error logging to the Next.js app preparation process in server.js, including guidance for missing production builds. In versionRouter, always fetches the latest update.sh from GitHub before running updates, logging the outcome and falling back to the local script if fetching fails.

Fixes: #442

✅ Prerequisites (X in brackets)

Self-review completed – Code follows project standards.
Tested thoroughly – Changes work as expected.
No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

Screenshot for frontend Change

🛠️ Type of Change (X in brackets)

🐞 Bug fix – Resolves an issue without breaking functionality.
✨ New feature – Adds new, non-breaking functionality.
💥 Breaking change – Alters existing functionality in a way that may require updates.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/community-scripts/ProxmoxVE-Local/pull/443 **Author:** [@MickLesk](https://github.com/MickLesk) **Created:** 1/13/2026 **Status:** ✅ Merged **Merged:** 1/13/2026 **Merged by:** [@MickLesk](https://github.com/MickLesk) **Base:** `main` ← **Head:** `get_update_script` --- ### 📝 Commits (1) - [`60f81fa`](https://github.com/community-scripts/ProxmoxVE-Local/commit/60f81fabc33da32559dcfe9faeab67cfae70af59) Improve server startup logging and update script fetching ### 📊 Changes **2 files changed** (+27 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `server.js` (+6 -0) 📝 `src/server/api/routers/version.ts` (+21 -0) </details> ### 📄 Description  ## ✍️ Description Adds success and error logging to the Next.js app preparation process in server.js, including guidance for missing production builds. In versionRouter, always fetches the latest update.sh from GitHub before running updates, logging the outcome and falling back to the local script if fetching fails. ## 🔗 Related PR / Issue Fixes: #442 ## ✅ Prerequisites (**X** in brackets) - [x] **Self-review completed** – Code follows project standards. - [x] **Tested thoroughly** – Changes work as expected. - [x] **No security risks** – No hardcoded secrets, unnecessary privilege escalations, or permission issues. ## Screenshot for frontend Change --- ## 🛠️ Type of Change (**X** in brackets) - [x] 🐞 **Bug fix** – Resolves an issue without breaking functionality. - [ ] ✨ **New feature** – Adds new, non-breaking functionality. - [ ] 💥 **Breaking change** – Alters existing functionality in a way that may require updates. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>