From 84c69ac014cb8ba0362ef2fcaa4ba38ce1f9ea20 Mon Sep 17 00:00:00 2001 From: Bradley Goulding Date: Thu, 15 May 2025 22:54:31 +0200 Subject: [PATCH] make env handling more robust --- composer-action.bash | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/composer-action.bash b/composer-action.bash index 23f6e2d..6be9a87 100755 --- a/composer-action.bash +++ b/composer-action.bash @@ -189,13 +189,18 @@ done <<<$(docker run --rm "${docker_tag}" env) while IFS= read -r line do - key=$(echo "$line" | cut -f1 -d=) - if printf '%s\n' "${dockerKeys[@]}" | grep -q -P "^${key}\$" - then - echo "Skipping env variable $key" >> output.log - else - echo "$line" >> DOCKER_ENV - fi + # Only process lines that look like valid KEY=VALUE pairs + if [[ "$line" =~ ^[A-Za-z_][A-Za-z0-9_]*= ]]; then + key=$(echo "$line" | cut -f1 -d=) + if printf '%s\n' "${dockerKeys[@]}" | grep -q -P "^${key}\$" + then + echo "Skipping env variable $key" >> output.log + else + echo "$line" >> DOCKER_ENV + fi + else + echo "Skipping invalid env line: $line" >> output.log + fi done <<<$(env) if [ -n "$ACTION_CONTAINER_WORKDIR" ]; then