From 91455f3f4117ae06ecc2d2a931b81451fa58dee2 Mon Sep 17 00:00:00 2001
From: duffyduck <info@hacker-net.de>
Date: Sun, 20 Sep 2026 22:29:26 +0200
Subject: [PATCH] Zugangsdaten pruefen: ohne den Umweg ueber Edition und
 Version

Der Menuepunkt fragte erst nach Edition, dann nach Version, dann ob die
Zugangsdaten aus der .env genommen werden sollen - und "image test"
holt sich Edition und Version ohnehin von dort. Derselbe Weg zum
zweiten Mal, fuer eine Frage, die nur lautet: taugen die hinterlegten
Zugangsdaten?

Jetzt laeuft die Pruefung direkt los und zeigt das Ergebnis. Nur wenn
in der .env ueberhaupt keine Zugangsdaten stehen, wird einmal danach
gefragt - dort hat ein Test ohne Eingabe keinen Sinn. Gespeichert wird
dabei nichts; das gehoert in die Einstellungen.

Auf der Kommandozeile bleibt alles wie es war: "image test --edition X
--version Y" nimmt weiterhin abweichende Angaben.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 scripts/cmk-manage.sh | 34 ++++++++++++++++++++++++----------
 1 file changed, 24 insertions(+), 10 deletions(-)

diff --git a/scripts/cmk-manage.sh b/scripts/cmk-manage.sh
index c85d0af..273a1e7 100755
--- a/scripts/cmk-manage.sh
+++ b/scripts/cmk-manage.sh
@@ -6141,20 +6141,34 @@ gui_image_versions() {
     rm -f "$f"
 }
 
+# Hier geht es nur um eine Frage: taugen die hinterlegten Zugangsdaten?
+# Edition und Version nimmt "image test" ohnehin aus der .env - danach
+# noch einmal zu fragen, waere derselbe Weg zum zweiten Mal.
 gui_image_test() {
-    local ed ver user pass
-    ed="$(gui_image_pick_edition)" || return 0
-    ver="$(ui_input "Version" "Version (fuer die Pruefung des Downloadportals):" "${CMK_VERSION:-}")" || return 0
     local -a extra=()
-    if ui_yesno "Zugangsdaten" "Zugangsdaten aus der .env verwenden?\n\nRegistry : ${CMK_REGISTRY_USER:-<leer>}\nDownload : ${CMK_DOWNLOAD_USER:-<leer>}\n\n<Nein> = jetzt eingeben"; then
-        :
-    else
-        user="$(ui_input "Zugangsdaten" "Benutzer (Checkmk-Kundenportal):" "${CMK_REGISTRY_USER:-}")" || return 0
+    if [ -z "${CMK_REGISTRY_USER:-}${CMK_DOWNLOAD_USER:-}" ]; then
+        ui_yesno "Zugangsdaten" "In der .env stehen keine Zugangsdaten.
+
+Jetzt welche eingeben und damit pruefen?
+Gespeichert werden sie dabei NICHT - das geht unter
+Einstellungen > Checkmk-Image und Zugangsdaten." || return 0
+        local user pass
+        user="$(ui_input "Zugangsdaten" "Benutzer (Checkmk-Kundenportal):" "")" || return 0
+        [ -n "$user" ] || return 0
         pass="$(ui_password "Zugangsdaten" "Kennwort:")" || return 0
-        [ -n "$user" ] && extra+=(--user "$user" --password "$pass")
+        extra=(--user "$user" --password "$pass")
     fi
-    ui_run "Zugangsdaten pruefen" bash -c \
-        "cd '$PROJECT_DIR' && NO_COLOR=1 '$SELF' $SELF_ENV image test --edition '$ed' --version '$ver' ${extra[*]+${extra[*]}}"
+
+    local f rc=0
+    f="$(mktemp)"
+    ( NO_COLOR=1 image_cmd_test ${extra[@]+"${extra[@]}"} ) >"$f" 2>&1 || rc=$?
+    if [ "$rc" -eq 0 ]; then
+        ui_text "Zugangsdaten - in Ordnung" "$f"
+    else
+        ui_text "Zugangsdaten - Pruefung fehlgeschlagen" "$f"
+    fi
+    rm -f "$f"
+    return 0
 }
 
 gui_image_pull() {