Skip to content

Commit bacf880

Browse files
author
eliranb
committed
fix update_config function
1 parent ef9c70e commit bacf880

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

lightrun-init-agent/update_config.sh

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -68,29 +68,28 @@ merge_configs() {
6868
update_config() {
6969
echo "Updating configuration with environment variables"
7070
local config_file="${WORK_DIR}/agent.config"
71+
local missing_configuration_params=""
7172

7273
if sed -n "s|com.lightrun.server=.*|com.lightrun.server=https://${LIGHTRUN_SERVER}|p" "${config_file}" | grep -q .; then
7374
# Perform actual in-place change
7475
sed -i "s|com.lightrun.server=.*|com.lightrun.server=https://${LIGHTRUN_SERVER}|" "${config_file}"
75-
exit 0
7676
else
77-
echo "No match found"
78-
exit 1
77+
missing_configuration_params="${missing_configuration_params} com.lightrun.server"
7978
fi
8079
if sed -n "s|com.lightrun.secret=.*|com.lightrun.secret=${LIGHTRUN_KEY}|p" "${config_file}" | grep -q .; then
8180
# Perform actual in-place change
8281
sed -i "s|com.lightrun.secret=.*|com.lightrun.secret=${LIGHTRUN_KEY}|" "${config_file}"
83-
exit 0
8482
else
85-
echo "No match found"
86-
exit 1
83+
missing_configuration_params="${missing_configuration_params} com.lightrun.secret"
8784
fi
8885
if sed -n "s|pinned_certs=.*|pinned_certs=${PINNED_CERT}|p" "${config_file}" | grep -q .; then
8986
# Perform actual in-place change
9087
sed -i "s|pinned_certs=.*|pinned_certs=${PINNED_CERT}|" "${config_file}"
91-
exit 0
9288
else
93-
echo "No match found"
89+
missing_configuration_params="${missing_configuration_params} pinned_certs"
90+
fi
91+
if [ -n "${missing_configuration_params}" ]; then
92+
echo "Error: Missing configuration parameters:${missing_configuration_params}"
9493
exit 1
9594
fi
9695
}

0 commit comments

Comments
 (0)