Skip to content

Commit fe4ae48

Browse files
committed
Only check for new users
1 parent 06c2e58 commit fe4ae48

File tree

3 files changed

+15
-2
lines changed

3 files changed

+15
-2
lines changed

bin/v-add-user

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@ name=$5
2121
if [ -n "$6" ]; then
2222
name="$name $6"
2323
fi
24+
25+
FROM_V_ADD_USER=true
26+
2427
# Includes
2528
# shellcheck source=/etc/hestiacp/hestia.conf
2629
source /etc/hestiacp/hestia.conf

func/main.sh

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -748,8 +748,12 @@ is_user_format_valid() {
748748
if [ "$1" != "${1//[^[:ascii:]]/}" ]; then
749749
check_result "$E_INVALID" "invalid $2 format :: $1"
750750
fi
751-
if ! [[ "$1" =~ ^[a-zA-Z]+ ]]; then
752-
check_result "$E_INVALID" "invalid $2 format :: $1"
751+
752+
# Only for new users
753+
if [[ "$FROM_V_ADD_USER" == "true" ]]; then
754+
if ! [[ "$1" =~ ^[a-zA-Z][-|.|_[:alnum:]]{0,28}[[:alnum:]]$ ]]; then
755+
check_result "$E_INVALID" "invalid $2 format :: $1"
756+
fi
753757
fi
754758
}
755759

test/test.bats

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -390,6 +390,12 @@ function check_ip_not_banned(){
390390
assert_output --partial 'Error: invalid user format'
391391
}
392392

393+
@test "User: Add new user Failed 5" {
394+
run v-add-user '1aap' $user $user@hestiacp2.com default "Super Test"
395+
assert_failure $E_INVALID
396+
assert_output --partial 'Error: invalid user format'
397+
}
398+
393399
@test "User: Add new user Success 1" {
394400
run v-add-user 'jaap01' $user $user@hestiacp2.com default "Super Test"
395401
assert_success

0 commit comments

Comments
 (0)