Refactor user existence checks in MinIO scripts to utilize JSON output. Improved error handling in healthcheck script and streamlined access key verification in entrypoint script.

Author: jaydrogers

src/entrypoint.sh

@@ -16,12 +16,11 @@ fi
 ################################################################################
 
 check_user_exists() {
-    local user_list
-    user_list=$(mc admin user ls "$MINIO_ALIAS")
-    case "$user_list" in
-        *"$MINIO_USER_ACCESS_KEY"*) return 0 ;;
-        *) return 1 ;;
-    esac
+    local user_json
+    user_json=$(mc admin user ls "$MINIO_ALIAS" --json)
+    user_json=${user_json#*\"accessKey\":\"}
+    local access_key=${user_json%%\"*}
+    [ "$access_key" = "$MINIO_USER_ACCESS_KEY" ]
 }
 
 create_policy() {

src/healthcheck.sh

@@ -11,16 +11,16 @@ echo "Checking for policy: $MINIO_POLICY_NAME"
 
 # Check if user exists
 echo "Checking MinIO user..."
-user_list=$(mc admin user ls "$MINIO_ALIAS")
-case "$user_list" in
-    *"$MINIO_USER_ACCESS_KEY"*)
-        echo "✅ MinIO user found"
-        ;;
-    *)
-        echo "ERROR: MinIO user $MINIO_USER_ACCESS_KEY not found"
-        exit 1
-        ;;
-esac
+user_json=$(mc admin user ls "$MINIO_ALIAS" --json)
+# Extract accessKey from JSON using bash string manipulation
+user_json=${user_json#*\"accessKey\":\"}
+access_key=${user_json%%\"*}
+if [ "$access_key" = "${MINIO_USER_ACCESS_KEY}" ]; then
+    echo "✅ MinIO user found"
+else
+    echo "ERROR: MinIO user ${MINIO_USER_ACCESS_KEY} not found"
+    exit 1
+fi
 
 # Check if policy exists
 echo "Checking MinIO policy..."