AhmedTawfik/planka
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Add name length validation to admin user seeding process

Browse Source
This commit is contained in:
Ayman
2026-07-08 07:17:14 +01:00
parent 1d523cdab5
commit 062cc3c347
2 changed files with 13 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
server/db/create-admin-user.js
+8
View File
@@ -60,6 +60,14 @@ const input = async (fieldName, options = {}) => {
isRequired: true,
});
// eslint-disable-next-line no-constant-condition
while (process.env.DEFAULT_ADMIN_NAME.length > 128) {
console.log('Name must be no more than 128 characters.');
process.env.DEFAULT_ADMIN_NAME = await input('Name', {
isRequired: true,
});
}
process.env.DEFAULT_ADMIN_USERNAME = await input('Username');
if (process.env.DEFAULT_ADMIN_USERNAME) {
server/db/seeds/default.js
+5
View File
@@ -20,8 +20,13 @@ const buildUserData = () => {
data.password = bcrypt.hashSync(process.env.DEFAULT_ADMIN_PASSWORD, 10);
}
if (process.env.DEFAULT_ADMIN_NAME) {
if (process.env.DEFAULT_ADMIN_NAME.length > 128) {
console.warn('Warning: DEFAULT_ADMIN_NAME exceeds 128 characters; truncating.');
data.name = process.env.DEFAULT_ADMIN_NAME.slice(0, 128);
} else {
data.name = process.env.DEFAULT_ADMIN_NAME;
}
}
if (process.env.DEFAULT_ADMIN_USERNAME) {
const username = process.env.DEFAULT_ADMIN_USERNAME.toLowerCase();