Merge pull request #302 from Vaso73/fix/pve-update-failure-status

fix: report Proxmox update failures correctly

scripts/global/update-pve-safe.sh

@@ -104,26 +104,26 @@ update_pve_safe() {
         return 1
     fi
 
-    # Reachability check: HEAD https://download.proxmox.com over the same
+    # Reachability check: probe the public Proxmox repository over the
-    # transport apt-get update will use (HTTPS 443). Previously a single
+    # transport apt is most likely to use. Many PVE installs use the
-    # ICMP ping — hosts behind firewalls that filter ICMP but allow 443
+    # official HTTP apt URI, while HTTPS may fail before apt ever runs
-    # (typical corporate / cloud-provider setups) hit a false negative
+    # if the CDN presents a certificate for another Proxmox hostname.
-    # and the update aborted even though the repository was reachable.
+    # Accept either transport and let apt-get update report repo-specific
-    # Two attempts with a short pause absorb transient network glitches
+    # errors in the next step.
-    # without adding perceptible latency when the network is healthy.
     _repo_reachable() {
-        local url="https://download.proxmox.com/"
+        local url attempt
-        local attempt
+        for url in "http://download.proxmox.com/" "https://download.proxmox.com/"; do
             for attempt in 1 2; do
                 if curl -sfI --connect-timeout 5 --max-time 10 -o /dev/null "$url"; then
                     return 0
                 fi
                 [[ $attempt -eq 1 ]] && sleep 1
             done
+        done
         return 1
     }
     if ! _repo_reachable; then
-        msg_error "$(translate "Cannot reach https://download.proxmox.com (HTTPS 443). Check network, proxy or DNS.")"
+        msg_error "$(translate "Cannot reach download.proxmox.com. Check network, proxy or DNS.")"
         echo -e
         msg_success "$(translate "Press Enter to return to menu...")"
         read -r

scripts/utilities/proxmox_update.sh

@@ -86,6 +86,8 @@ apt_upgrade() {
     # Single worker for both PVE 8 and 9 — it detects the version itself
     # and only performs operations safe on a production host.
     bash "$LOCAL_SCRIPTS/global/update-pve-safe.sh"
+    local worker_rc=$?
+    return "$worker_rc"
 
 
 }
@@ -162,9 +164,12 @@ check_reboot() {
 
 
 apt_upgrade
+update_rc=$?
+if [[ "$update_rc" -eq 0 ]]; then
     check_reboot
-
+else
-    
+    exit "$update_rc"
+fi