From c137296dcdbc0e6663a14afc3601029635803353 Mon Sep 17 00:00:00 2001 From: CyberMind-FR Date: Wed, 25 Feb 2026 14:37:33 +0100 Subject: [PATCH] fix(voipctl): Fix JSON output in status command - Fix active_calls and extensions count to output clean integers - Remove tr -cd which was causing duplicate values in JSON - Use simpler variable assignment with fallback to 0 - Prevents malformed JSON output from cmd_status() Co-Authored-By: Claude Opus 4.5 --- .../secubox-app-voip/files/usr/sbin/voipctl | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/package/secubox/secubox-app-voip/files/usr/sbin/voipctl b/package/secubox/secubox-app-voip/files/usr/sbin/voipctl index 46fc7f78..640d61c1 100755 --- a/package/secubox/secubox-app-voip/files/usr/sbin/voipctl +++ b/package/secubox/secubox-app-voip/files/usr/sbin/voipctl @@ -284,17 +284,21 @@ cmd_status() { if container_running; then running=1 - + # Check trunk registration if container_exec asterisk -rx "pjsip show registrations" 2>/dev/null | grep -q "Registered"; then registered=1 fi - + # Count active calls - active_calls=$(container_exec asterisk -rx "core show channels" 2>/dev/null | grep -oE "^[0-9]+ active" | cut -d' ' -f1 || echo 0) - + local calls_output + calls_output=$(container_exec asterisk -rx "core show channels" 2>/dev/null | grep -oE "^[0-9]+ active" | head -1 | cut -d' ' -f1) || true + active_calls=${calls_output:-0} + # Count extensions - extensions=$(container_exec asterisk -rx "pjsip show endpoints" 2>/dev/null | grep -c "^[0-9]" || echo 0) + local ext_output + ext_output=$(container_exec asterisk -rx "pjsip show endpoints" 2>/dev/null | wc -l) || true + extensions=${ext_output:-0} fi cat <