Errors highlighted by shellcheck for pam-eap-setup
garethahealy opened this issue · comments
Gareth Healy commented
@erouvas ; be good to fix or ignore these via the rules provided by shellcheck
find . -type f -name "*.sh" -exec shellcheck --severity=error {} \;
In ./pam-eap-setup/pam-setup.sh line 494:
for i in ${controllerListAr[@]}; do
^--------------------^ SC2068: Double quote array expansions to avoid re-splitting elements.
In ./pam-eap-setup/pam-setup.sh line 886:
summary "Using Controller List :- ${controllerListAr[@]}"
^--------------------^ SC2145: Argument mixes string and array. Use * or separate argument.
In ./pam-eap-setup/pam-setup.sh line 902:
for ondx in ${!multiOptions[@]}; do
^-----------------^ SC2068: Double quote array expansions to avoid re-splitting elements.
In ./pam-eap-setup/pam-setup.sh line 917:
for eap_patch_file in `ls $EAP_PATCH_ZIP 2> /dev/null`; do
^-- SC2045: Iterating over ls output is fragile. Use globs.
In ./pam-eap-setup/pam-setup.sh line 1003:
jvm_memory=${configOptions[jvm_memory]} && [[ ! -z "jvm_memory" ]] && nodeConfig['jvm_memory']="$jvm_memory"
^--------^ SC2157: Argument to -z is always false due to literal strings.
highlighted via this PR:
Gareth Healy commented