Fix user handle naming logic (#4969) * Fix user handle naming logic * Extract slugify logic into a module func --------- Co-authored-by: Cohee <18619528+Cohee1207@users.noreply.github.com>

f6a2def4037789551b8318afde5a0f22cf517431

allen9441 <72611024+allen9441@users.noreply.github.com>

Signed

src/endpoints/users-admin.js+16 -2 Viewed

 export const router = express.Router();
+/**
+ * Slugifies a given text string.
+ * - Converts to lowercase
+ * - Trims whitespace
+ * - Replaces spaces and special characters with hyphens
+ * - Removes leading and trailing hyphens
+ * - Uses lodash.deburr to remove diacritical marks
+ * @param {string} text Text to slugify
+ * @returns {string} Slugified text
+ */
+function slugify(text) {
+    return lodash.deburr(String(text ?? '').toLowerCase().trim()).replace(/[^a-z0-9]+/g, '-').replace(/^-+|-+$/g, '');
+}
 router.post('/get', requireAdminMiddleware, async (_request, response) => {
     try {
         /** @type {import('../users.js').User[]} */

+        }
         const handles = await getAllUserHandles();
-        const handle = lodash.kebabCase(String(request.body.handle).toLowerCase().trim());
+        const handle = slugify(request.body.handle);
         if (!handle) {
             console.warn('Create user failed: Invalid handle');

             return response.status(400).json({ error: 'Missing required fields' });
+        }
-        const text = lodash.kebabCase(String(request.body.text).toLowerCase().trim());
+        const text = slugify(request.body.text);
         return response.send(text);
     } catch (error) {