[FIX] Generate unique slug for user registration
- Implement slug uniqueness check before creating user - Add numeric suffix if slug already exists (e.g., username1, username2) - Fallback to timestamp-based slug if too many collisions - Prevents database constraint violations for duplicate slugs - Matches the logic used in OAuth service for consistency
This commit is contained in:
parent
1011ddd4a3
commit
e63a0f2720
1 changed files with 21 additions and 1 deletions
|
|
@ -113,12 +113,32 @@ func (s *AuthService) Register(ctx context.Context, email, username, password st
|
|||
return nil, err
|
||||
}
|
||||
|
||||
// Générer un slug unique à partir du username
|
||||
// Le slug doit être unique, donc on vérifie et on ajoute un suffixe si nécessaire
|
||||
baseSlug := strings.ToLower(username)
|
||||
slug := baseSlug
|
||||
counter := 1
|
||||
for {
|
||||
var count int64
|
||||
err := s.db.WithContext(ctx).Model(&models.User{}).Where("slug = ?", slug).Count(&count).Error
|
||||
if err == nil && count == 0 {
|
||||
break
|
||||
}
|
||||
slug = fmt.Sprintf("%s%d", baseSlug, counter)
|
||||
counter++
|
||||
if counter > 1000 {
|
||||
// Fallback: utiliser un timestamp si trop de collisions
|
||||
slug = fmt.Sprintf("user_%d", time.Now().Unix())
|
||||
break
|
||||
}
|
||||
}
|
||||
|
||||
// Créer l'utilisateur dans la base de données
|
||||
user := &models.User{
|
||||
ID: uuid.New(), // Générer un nouvel UUID
|
||||
Email: email,
|
||||
Username: username,
|
||||
Slug: strings.ToLower(username),
|
||||
Slug: slug,
|
||||
PasswordHash: string(hashedPassword),
|
||||
Role: "user", // Valeur par défaut
|
||||
IsActive: true, // Valeur par défaut
|
||||
|
|
|
|||
Loading…
Reference in a new issue