Merge pull request #153 from openbase/fix_bco_option_handling_in_docker_entry_point

fix entry point multi argument handling for docker.

Author: LukBelter

docker/docker-entrypoint.sh

@@ -15,11 +15,24 @@ if [ -z ${BCO_MODULE_PREPARE_SCRIPT+x} ]; then
     echo "no module preperation required.";
 else
     echo "prepare bco modules...";
-    source /usr/local/bin/${BCO_MODULE_PREPARE_SCRIPT};
+    # shellcheck disable=SC1090  # source a dynamic script path intentionally
+    source /usr/local/bin/"${BCO_MODULE_PREPARE_SCRIPT}";
 fi
 
 # replace the current pid 1 with original entrypoint
-echo "start main application: $@"
+# Use printf to safely print all positional args without mixing scalar and array expansions
+printf 'start main application: %s\n' "$*"
 
-set -- "$@" --bco-home "${BCO_HOME}" --log-dir "${BCO_LOGS}" --host "${MQTT_BROKER}" "${BCO_OPTIONS}"
+# If BCO_OPTIONS contains multiple space-separated arguments, expand them as distinct
+# arguments by splitting into a bash array. This avoids passing the entire string as
+# a single argument when using "${BCO_OPTIONS}".
+_bco_options_array=()
+if [ -n "${BCO_OPTIONS:-}" ]; then
+    # read -a splits on IFS (whitespace), preserving each option as a separate element
+    # Note: if options themselves need to contain spaces, provide them via an array env
+    # or a different mechanism; this simple split handles the common case of multiple flags.
+    read -r -a _bco_options_array <<< "${BCO_OPTIONS}"
+fi
+
+set -- "$@" --bco-home "${BCO_HOME}" --log-dir "${BCO_LOGS}" --host "${MQTT_BROKER}" "${_bco_options_array[@]}"
 exec "$@"

lib/jul

@@ -76,9 +76,9 @@ version.org.openbase..jul.communication.mqtt.test=3.7.2
 
 version.org.openbase..jul.transformation=3.7.2
 
-version.org.testcontainers..junit-jupiter=1.18.3
+version.org.testcontainers..junit-jupiter=1.21.4
 
-version.org.testcontainers..testcontainers=1.18.3
+version.org.testcontainers..testcontainers=2.0.3
 
 version.org.springframework.boot..spring-boot-starter-webflux=3.1.2