[bugfix] add in-use checks for admin cli account creation (#904)

This commit is contained in:
tobi 2022-10-11 12:16:34 +02:00 committed by GitHub
parent 832befd727
commit 5cd087241b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -46,6 +46,14 @@ var Create action.GTSAction = func(ctx context.Context) error {
return err
}
usernameAvailable, err := dbConn.IsUsernameAvailable(ctx, username)
if err != nil {
return err
}
if !usernameAvailable {
return fmt.Errorf("username %s is already in use", username)
}
email := config.GetAdminAccountEmail()
if email == "" {
return errors.New("no email set")
@ -54,6 +62,14 @@ var Create action.GTSAction = func(ctx context.Context) error {
return err
}
emailAvailable, err := dbConn.IsEmailAvailable(ctx, email)
if err != nil {
return err
}
if !emailAvailable {
return fmt.Errorf("email address %s is already in use", email)
}
password := config.GetAdminAccountPassword()
if password == "" {
return errors.New("no password set")