[feat] prefer name over email substring

jordan-dalby · Jan 5, 2025 · 76a1229 · 76a1229
1 parent 8adc608
commit 76a1229
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/server/src/repositories/userRepository.js b/server/src/repositories/userRepository.js
@@ -129,9 +129,17 @@ class UserRepository {
       const user = this.findByOIDCIdStmt.get(profile.sub, provider);
       if (user) return user;
 
-      const baseUsername = profile.email?.split('@')[0] || 
-                          profile.preferred_username ||
-                          profile.sub;
+      const sanitizeName = (name) => {
+        return name
+          .toLowerCase()
+          .replace(/[^a-z0-9]/g, '')
+          .slice(0, 30);
+      };
+
+      let baseUsername = profile.name ? sanitizeName(profile.name) :
+                        profile.preferred_username ? sanitizeName(profile.preferred_username) :
+                        profile.email?.split('@')[0] || 
+                        profile.sub;
 
       const username = await this.generateUniqueUsername(baseUsername);