Add proper name valoidation in rename command, don't allow blanks in names

This commit is contained in:
Jose
2020-05-09 13:59:58 -04:00
parent a607dc2719
commit e07f6cb0ed
2 changed files with 9 additions and 7 deletions

View File

@@ -445,8 +445,8 @@ else
fi fi
## don't allow for dots(.) in container names ## don't allow for dots(.) in container names
if echo "${NAME}" | grep -q "[.]"; then if echo "${NAME}" | grep -Eq '[.]|\ '; then
echo -e "${COLOR_RED}Container names may not contain a dot(.)!${COLOR_RESET}" echo -e "${COLOR_RED}Container names may not contain a dot(.) nor spaces!${COLOR_RESET}"
exit 1 exit 1
fi fi

View File

@@ -57,8 +57,8 @@ TARGET="${1}"
NEWNAME="${2}" NEWNAME="${2}"
shift shift
if echo "${NEWNAME}" | grep -q "[.]"; then if echo "${NEWNAME}" | grep -Eq '[.]|\ '; then
echo -e "${COLOR_RED}Container names may not contain a dot(.)!${COLOR_RESET}" echo -e "${COLOR_RED}Container names may not contain a dot(.) nor spaces!${COLOR_RESET}"
exit 1 exit 1
fi fi
@@ -125,9 +125,11 @@ change_name() {
fi fi
} }
# Check if container is running ## check if a running jail matches name or already exist
if [ -n "$(jls name | awk "/^${TARGET}$/")" ]; then if [ "$(jls name | awk "/^${TARGET}$/")" ]; then
error_notify "${COLOR_RED}${TARGET} is running, See 'bastille stop'.${COLOR_RESET}" error_notify "${COLOR_RED}Warning: ${TARGET} is running or the name does match.${COLOR_RESET}"
elif [ -d "${bastille_jailsdir}/${NEWNAME}" ]; then
error_notify "${COLOR_RED}Jail: ${NEWNAME} already exist.${COLOR_RESET}"
fi fi
change_name change_name