Compare commits

...

2 Commits

Author SHA1 Message Date
Chamika J
210a9a7aba fix(labels-controller): update color validation to use WorklenzColorShades for label updates
- Modified color validation logic in `updateLabel` method to check against `WorklenzColorShades` instead of `WorklenzColorCodes`.
- Ensured that the color input is validated correctly during label updates, enhancing data integrity.
2025-08-04 15:28:31 +05:30
Chamika J
6e37208f62 fix(password-validator): adjust password length validation to include maximum length of 32 characters 2025-08-04 15:23:58 +05:30
2 changed files with 4 additions and 4 deletions

View File

@@ -5,7 +5,7 @@ import db from "../config/db";
import {ServerResponse} from "../models/server-response"; import {ServerResponse} from "../models/server-response";
import WorklenzControllerBase from "./worklenz-controller-base"; import WorklenzControllerBase from "./worklenz-controller-base";
import HandleExceptions from "../decorators/handle-exceptions"; import HandleExceptions from "../decorators/handle-exceptions";
import {TASK_PRIORITY_COLOR_ALPHA, WorklenzColorCodes} from "../shared/constants"; import {TASK_PRIORITY_COLOR_ALPHA, WorklenzColorCodes, WorklenzColorShades} from "../shared/constants";
export default class LabelsController extends WorklenzControllerBase { export default class LabelsController extends WorklenzControllerBase {
@HandleExceptions() @HandleExceptions()
@@ -73,7 +73,7 @@ export default class LabelsController extends WorklenzControllerBase {
WHERE id = $1 WHERE id = $1
AND team_id = $2;`; AND team_id = $2;`;
if (!WorklenzColorCodes.includes(req.body.color)) if (!Object.values(WorklenzColorShades).flat().includes(req.body.color))
return res.status(400).send(new ServerResponse(false, null)); return res.status(400).send(new ServerResponse(false, null));
const result = await db.query(q, [req.params.id, req.user?.team_id, req.body.color]); const result = await db.query(q, [req.params.id, req.user?.team_id, req.body.color]);
@@ -92,7 +92,7 @@ export default class LabelsController extends WorklenzControllerBase {
} }
if (req.body.color) { if (req.body.color) {
if (!WorklenzColorCodes.includes(req.body.color)) if (!Object.values(WorklenzColorShades).flat().includes(req.body.color))
return res.status(400).send(new ServerResponse(false, null)); return res.status(400).send(new ServerResponse(false, null));
updates.push(`color_code = $${paramIndex++}`); updates.push(`color_code = $${paramIndex++}`);
values.push(req.body.color); values.push(req.body.color);

View File

@@ -8,7 +8,7 @@ import {PASSWORD_POLICY} from "../../shared/constants";
function isStrongPassword(password: string) { function isStrongPassword(password: string) {
if (!isProduction()) return true; if (!isProduction()) return true;
const strength = PasswordStrengthChecker.validate(password); const strength = PasswordStrengthChecker.validate(password);
return strength.value >= 2 && strength.length < 32; return strength.value >= 2 && strength.length <= 32;
} }
export default function (req: Request, res: Response, next: NextFunction) { export default function (req: Request, res: Response, next: NextFunction) {