From cfb4f7e6d0da4e17e4c8a6ef2a25fc603755b58d Mon Sep 17 00:00:00 2001 From: Gui-Gos <97973228+Gui-Gos@users.noreply.github.com> Date: Sun, 4 Jan 2026 13:19:45 +0100 Subject: [PATCH] fix: Resolve shellcheck SC2155 warnings in health check script MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Separate variable declaration and assignment to avoid masking return values. This follows shellcheck best practices for error handling. Changes: - Separate declaration/assignment for container_name (line 59-60) - Separate declaration/assignment for status (line 67-69) - Separate declaration/assignment for health (line 68-70) - Separate declaration/assignment for exit_code (line 86-87) All 4 SC2155 warnings are now resolved. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 --- apps/wazuh-runtipi/data/debug/wazuh-health-check.sh | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/apps/wazuh-runtipi/data/debug/wazuh-health-check.sh b/apps/wazuh-runtipi/data/debug/wazuh-health-check.sh index b310a83..8f25022 100644 --- a/apps/wazuh-runtipi/data/debug/wazuh-health-check.sh +++ b/apps/wazuh-runtipi/data/debug/wazuh-health-check.sh @@ -56,15 +56,18 @@ print_section() { # Function to check service check_service() { local service=$1 - local container_name=$(docker ps -a --format '{{.Names}}' | grep -E "${WAZUH_PREFIX}.*${service}" | head -1) + local container_name + container_name=$(docker ps -a --format '{{.Names}}' | grep -E "${WAZUH_PREFIX}.*${service}" | head -1) if [ -z "$container_name" ]; then echo -e "${RED}✗ Container not found${NC}" return 1 fi - local status=$(docker inspect --format='{{.State.Status}}' "$container_name" 2>/dev/null) - local health=$(docker inspect --format='{{.State.Health.Status}}' "$container_name" 2>/dev/null || echo "no healthcheck") + local status + local health + status=$(docker inspect --format='{{.State.Status}}' "$container_name" 2>/dev/null) + health=$(docker inspect --format='{{.State.Health.Status}}' "$container_name" 2>/dev/null || echo "no healthcheck") echo -ne " $service: " @@ -80,7 +83,8 @@ check_service() { return 1 fi elif [ "$status" = "exited" ]; then - local exit_code=$(docker inspect --format='{{.State.ExitCode}}' "$container_name" 2>/dev/null) + local exit_code + exit_code=$(docker inspect --format='{{.State.ExitCode}}' "$container_name" 2>/dev/null) if [ "$exit_code" = "0" ]; then echo -e "${GREEN}✓ Exited successfully (code 0)${NC}" return 0