mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2025-02-21 13:26:24 -05:00
46e60ce966
- The doctor commands to check the validity of existing usernames and email addresses depend on functionality that have configurable behavior depending on the values of the `[service]` settings, so load them when running the doctor command. - Resolves #6664 - No unit test due to the architecture of doctor commands. # Testing 1. Set `[service].ALLOW_DOTS_IN_USERNAMES = true`. 2. Create a user that contains a dot in their username. 3. Run the `check-user-name` doctor command and verify there's no error reported. Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/6674 Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org> Co-authored-by: Gusted <postmaster@gusted.xyz> Co-committed-by: Gusted <postmaster@gusted.xyz>
103 lines
3 KiB
Go
103 lines
3 KiB
Go
// Copyright 2022 The Gitea Authors. All rights reserved.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package doctor
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
|
|
"code.gitea.io/gitea/models/db"
|
|
"code.gitea.io/gitea/models/user"
|
|
"code.gitea.io/gitea/modules/log"
|
|
"code.gitea.io/gitea/modules/setting"
|
|
"code.gitea.io/gitea/modules/validation"
|
|
|
|
"xorm.io/builder"
|
|
)
|
|
|
|
func iterateUserAccounts(ctx context.Context, each func(*user.User) error) error {
|
|
err := db.Iterate(
|
|
ctx,
|
|
builder.Gt{"id": 0},
|
|
func(ctx context.Context, bean *user.User) error {
|
|
return each(bean)
|
|
},
|
|
)
|
|
return err
|
|
}
|
|
|
|
// Since 1.16.4 new restrictions has been set on email addresses. However users with invalid email
|
|
// addresses would be currently facing a error due to their invalid email address.
|
|
// Ref: https://github.com/go-gitea/gitea/pull/19085 & https://github.com/go-gitea/gitea/pull/17688
|
|
func checkUserEmail(ctx context.Context, logger log.Logger, _ bool) error {
|
|
setting.LoadServiceSetting()
|
|
|
|
// We could use quirky SQL to get all users that start without a [a-zA-Z0-9], but that would mean
|
|
// DB provider-specific SQL and only works _now_. So instead we iterate through all user accounts
|
|
// and use the validation.ValidateEmail function to be future-proof.
|
|
var invalidUserCount int64
|
|
if err := iterateUserAccounts(ctx, func(u *user.User) error {
|
|
// Only check for users, skip
|
|
if u.Type != user.UserTypeIndividual {
|
|
return nil
|
|
}
|
|
|
|
if err := validation.ValidateEmail(u.Email); err != nil {
|
|
invalidUserCount++
|
|
logger.Warn("User[id=%d name=%q] have not a valid e-mail: %v", u.ID, u.Name, err)
|
|
}
|
|
return nil
|
|
}); err != nil {
|
|
return fmt.Errorf("iterateUserAccounts: %w", err)
|
|
}
|
|
|
|
if invalidUserCount == 0 {
|
|
logger.Info("All users have a valid e-mail.")
|
|
} else {
|
|
logger.Warn("%d user(s) have a non-valid e-mail.", invalidUserCount)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
// From time to time Gitea makes changes to the reserved usernames and which symbols
|
|
// are allowed for various reasons. This check helps with detecting users that, according
|
|
// to our reserved names, don't have a valid username.
|
|
func checkUserName(ctx context.Context, logger log.Logger, _ bool) error {
|
|
setting.LoadServiceSetting()
|
|
|
|
var invalidUserCount int64
|
|
if err := iterateUserAccounts(ctx, func(u *user.User) error {
|
|
if err := user.IsUsableUsername(u.Name); err != nil {
|
|
invalidUserCount++
|
|
logger.Warn("User[id=%d] does not have a valid username: %v", u.ID, err)
|
|
}
|
|
return nil
|
|
}); err != nil {
|
|
return fmt.Errorf("iterateUserAccounts: %w", err)
|
|
}
|
|
|
|
if invalidUserCount == 0 {
|
|
logger.Info("All users have a valid username.")
|
|
} else {
|
|
logger.Warn("%d user(s) have a non-valid username.", invalidUserCount)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
func init() {
|
|
Register(&Check{
|
|
Title: "Check if users has an valid email address",
|
|
Name: "check-user-email",
|
|
IsDefault: false,
|
|
Run: checkUserEmail,
|
|
Priority: 9,
|
|
})
|
|
Register(&Check{
|
|
Title: "Check if users have a valid username",
|
|
Name: "check-user-names",
|
|
IsDefault: false,
|
|
Run: checkUserName,
|
|
Priority: 9,
|
|
})
|
|
}
|