Merge pull request #845 from community-scripts/revert-844-codex/implement-centralized-logging-for-proxmoxve

Revert "feat: add centralized logging utilities"

misc/alpine-install.func

@@ -32,7 +32,6 @@ error_handler() {
   local exit_code="$?"
   local line_number="$1"
   local command="$2"
-  log_error "line $line_number: exit code $exit_code while executing command $command"
   local error_message="${RD}[ERROR]${CL} in line ${RD}$line_number${CL}: exit code ${RD}$exit_code${CL}: while executing command ${YW}$command${CL}"
   echo -e "\n$error_message\n"
 }

misc/api.func

@@ -2,16 +2,6 @@
 # Author: michelroegl-brunner
 # License: MIT | https://git.community-scripts.org/community-scripts/ProxmoxVED/raw/branch/main/LICENSE
 
-if [[ -f "$(dirname "${BASH_SOURCE[0]}")/logger.func" ]]; then
-  source "$(dirname "${BASH_SOURCE[0]}")/logger.func"
-else
-  if command -v curl >/dev/null 2>&1; then
-    source <(curl -fsSL https://raw.githubusercontent.com/community-scripts/ProxmoxVE/main/misc/logger.func)
-  elif command -v wget >/dev/null 2>&1; then
-    source <(wget -qO- https://raw.githubusercontent.com/community-scripts/ProxmoxVE/main/misc/logger.func)
-  fi
-fi
-
 get_error_description() {
   local exit_code="$1"
   case "$exit_code" in

misc/build.func

@@ -43,7 +43,6 @@ error_handler() {
   local line_number="$1"
   local command="$2"
   printf "\e[?25h"
-  log_error "line $line_number: exit code $exit_code while executing command $command"
   local error_message="[ERROR] in line $line_number: exit code $exit_code: while executing command $command"
   post_update_to_api "failed" "$command"
   echo -e "\n$error_message\n"

misc/core.func

@@ -9,16 +9,6 @@
 [[ -n "${_CORE_FUNC_LOADED:-}" ]] && return
 _CORE_FUNC_LOADED=1
 
-if [[ -f "$(dirname "${BASH_SOURCE[0]}")/logger.func" ]]; then
-  source "$(dirname "${BASH_SOURCE[0]}")/logger.func"
-else
-  if command -v curl >/dev/null 2>&1; then
-    source <(curl -fsSL https://raw.githubusercontent.com/community-scripts/ProxmoxVE/main/misc/logger.func)
-  elif command -v wget >/dev/null 2>&1; then
-    source <(wget -qO- https://raw.githubusercontent.com/community-scripts/ProxmoxVE/main/misc/logger.func)
-  fi
-fi
-
 load_functions() {
   [[ -n "${__FUNCTIONS_LOADED:-}" ]] && return
   __FUNCTIONS_LOADED=1
@@ -106,9 +96,9 @@ _tool_error_hint() {
 #   exit 143
 # }
 
-# logger.func now sets strict modes and traps globally
 catch_errors() {
-  :
+  set -Eeuo pipefail
+  trap 'error_handler $LINENO "$BASH_COMMAND"' ERR
 }
 
 # ------------------------------------------------------------------------------
@@ -349,7 +339,6 @@ stop_spinner() {
 msg_info() {
   local msg="$1"
   [[ -z "$msg" ]] && return
-  log_info "$msg"
 
   if ! declare -p MSG_INFO_SHOWN &>/dev/null || ! declare -A MSG_INFO_SHOWN &>/dev/null; then
     declare -gA MSG_INFO_SHOWN=()
@@ -376,7 +365,6 @@ msg_info() {
 msg_ok() {
   local msg="$1"
   [[ -z "$msg" ]] && return
-  log_info "$msg"
   stop_spinner
   clear_line
   printf "%s %b\n" "$CM" "${GN}${msg}${CL}" >&2
@@ -386,14 +374,12 @@ msg_ok() {
 msg_error() {
   stop_spinner
   local msg="$1"
-  log_error "$msg"
   echo -e "${BFR:-} ${CROSS:-✖️} ${RD}${msg}${CL}"
 }
 
 msg_warn() {
   stop_spinner
   local msg="$1"
-  log_warn "$msg"
   echo -e "${BFR:-} ${INFO:-ℹ️} ${YWB}${msg}${CL}"
 }
 
@@ -409,7 +395,6 @@ msg_custom() {
 function msg_debug() {
   if [[ "${var_full_verbose:-0}" == "1" ]]; then
     [[ "${var_verbose:-0}" != "1" ]] && var_verbose=1
-    log_debug "$*"
     echo -e "${YWB}[$(date '+%F %T')] [DEBUG]${CL} $*"
   fi
 }

misc/create_lxc.sh

@@ -36,7 +36,6 @@ function error_handler() {
   local line_number="$1"
   local command="$2"
   printf "\e[?25h"
-  log_error "line $line_number: exit code $exit_code while executing command $command"
   echo -e "\n${RD}[ERROR]${CL} in line ${RD}$line_number${CL}: exit code ${RD}$exit_code${CL}: while executing command ${YW}$command${CL}\n"
   exit "$exit_code"
 }

misc/install.func

@@ -36,7 +36,6 @@ error_handler() {
   local exit_code="$?"
   local line_number="$1"
   local command="$2"
-  log_error "line $line_number: exit code $exit_code while executing command $command"
   local error_message="${RD}[ERROR]${CL} in line ${RD}$line_number${CL}: exit code ${RD}$exit_code${CL}: while executing command ${YW}$command${CL}"
   echo -e "\n$error_message"
 

misc/logger.func

@@ -1,88 +0,0 @@
-#!/usr/bin/env bash
-# Centralized logging utilities for Proxmox helper scripts
-# Provides log_* functions, stdout/stderr capture, and robust error handling.
-
-# Prevent multiple sourcing
-[[ -n "${_LOGGER_FUNC_LOADED:-}" ]] && return
-_LOGGER_FUNC_LOADED=1
-
-# ------------------------------------------------------------------------------
-# Shell options – fail fast and propagate ERR through subshells
-# ------------------------------------------------------------------------------
-set -o errexit -o nounset -o pipefail -o errtrace
-
-# ------------------------------------------------------------------------------
-# Logfile preparation
-# ------------------------------------------------------------------------------
-LOGDIR=${LOGDIR:-/var/log/proxmoxve}
-mkdir -p "$LOGDIR" 2>/dev/null || true
-
-SCRIPT_NAME="${SCRIPT_NAME:-$(basename "$0")}";
-RUN_ID="${RUN_ID:-$(date +%Y%m%d_%H%M%S)_$$}";
-LOGFILE="${LOGFILE:-$LOGDIR/${SCRIPT_NAME%.sh}_$RUN_ID.log}"
-
-LOG_LEVEL="${LOG_LEVEL:-INFO}"
-declare -A LEVELS=([DEBUG]=0 [INFO]=1 [WARN]=2 [ERROR]=3)
-
-# Preserve original stdout/stderr for terminal output
-exec 3>&1 4>&2
-
-log_msg() {
-  local level="$1"; shift
-  local msg="$*"
-  local ts
-  ts="$(date '+%Y-%m-%d %H:%M:%S')"
-  echo "[$ts] [$SCRIPT_NAME] [$level] $msg" >>"$LOGFILE"
-  if (( ${LEVELS[$level]} >= ${LEVELS[$LOG_LEVEL]} )); then
-    case "$level" in
-      DEBUG)
-        [[ "${var_full_verbose:-0}" -eq 1 ]] && echo -e "\033[36m[DEBUG]\033[0m $msg" >&3 ;;
-      INFO)
-        echo -e "\033[34m[INFO]\033[0m  $msg" >&3 ;;
-      WARN)
-        echo -e "\033[33m[WARN]\033[0m  $msg" >&3 ;;
-      ERROR)
-        echo -e "\033[31m[ERROR]\033[0m $msg" >&4 ;;
-    esac
-  fi
-}
-
-log_debug() { log_msg DEBUG "$*"; }
-log_info()  { log_msg INFO  "$*"; }
-log_warn()  { log_msg WARN  "$*"; }
-log_error() { log_msg ERROR "$*"; }
-
-# Backward compatible wrappers
-msg_info()  { log_info  "ℹ️  $*"; }
-msg_ok()    { log_info  "✅ $*"; }
-msg_warn()  { log_warn  "⚠️  $*"; }
-msg_error() { log_error "❌ $*"; }
-msg_debug() { log_debug "$*"; }
-
-# ------------------------------------------------------------------------------
-# Capture arbitrary stdout/stderr (including from subshells)
-# ------------------------------------------------------------------------------
-log_stream() {
-  local level="$1"
-  while IFS= read -r line; do
-    log_msg "$level" "$line"
-  done
-}
-
-# Redirect script output through logger
-exec > >(log_stream INFO) 2> >(log_stream ERROR)
-
-# ------------------------------------------------------------------------------
-# Error handler – logs failing command, line, and exits
-# ------------------------------------------------------------------------------
-error_handler() {
-  local code="$?"
-  local cmd="${BASH_COMMAND:-unknown}"
-  local line="${BASH_LINENO[0]:-unknown}"
-  local file="${BASH_SOURCE[1]:-unknown}"
-  log_error "command '$cmd' failed in $file:$line with exit code $code"
-  exit "$code"
-}
-
-trap error_handler ERR
-