Skip to content

Commit 1bfdff6

Browse files
Merge pull request #57 from yash-pouranik/copilot/fix-auto-generated-username-length
fix: pad auto-generated username/name to minimum 3 characters
2 parents 28c480c + 265062f commit 1bfdff6

2 files changed

Lines changed: 13 additions & 4 deletions

File tree

apps/dashboard-api/src/controllers/userAuth.controller.js

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,17 @@ const buildAuthUserPayload = (usersColConfig, parsedData, hashedPassword, verifi
4242
}
4343

4444
if (hasRequiredField(usersColConfig, 'name') && (payload.name === undefined || payload.name === null || payload.name === '')) {
45-
payload.name = username || email.split('@')[0];
45+
const generatedName = username || email.split('@')[0];
46+
payload.name = generatedName.length >= 3 ? generatedName : generatedName.padEnd(3, '0');
4647
}
4748

4849
if (hasRequiredField(usersColConfig, 'username') && (payload.username === undefined || payload.username === null || payload.username === '')) {
49-
payload.username = payload.name || email.split('@')[0];
50+
const generatedUsernameBase =
51+
typeof payload.name === 'string' && payload.name !== ''
52+
? payload.name
53+
: email.split('@')[0];
54+
const generatedUsername = String(generatedUsernameBase);
55+
payload.username = generatedUsername.length >= 3 ? generatedUsername : generatedUsername.padEnd(3, '0');
5056
}
5157

5258
return payload;

apps/public-api/src/controllers/userAuth.controller.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,14 @@ const buildAuthUserPayload = (usersColConfig, parsedData, hashedPassword, verifi
4242
}
4343

4444
if (hasRequiredField(usersColConfig, 'name') && (payload.name === undefined || payload.name === null || payload.name === '')) {
45-
payload.name = username || email.split('@')[0];
45+
const generatedName = username || email.split('@')[0];
46+
payload.name = generatedName.length >= 3 ? generatedName : generatedName.padEnd(3, '0');
4647
}
4748

4849
if (hasRequiredField(usersColConfig, 'username') && (payload.username === undefined || payload.username === null || payload.username === '')) {
49-
payload.username = payload.name || email.split('@')[0];
50+
const baseUsername = typeof payload.name === 'string' ? payload.name : email.split('@')[0];
51+
const generatedUsername = baseUsername;
52+
payload.username = generatedUsername.length >= 3 ? generatedUsername : generatedUsername.padEnd(3, '0');
5053
}
5154

5255
return payload;

0 commit comments

Comments
 (0)