admin/bodyshop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feature/IO-3052-Skia-Canvas-Handler: Fix missing checks

Browse Source 
Signed-off-by: Dave Richer <dave@imexsystems.ca>
This commit is contained in:
Dave Richer
2024-12-05 12:16:32 -08:00
parent c84fbcaba1
commit 20bddb43b6
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
server/middleware/validateCanvasInputMiddleware.js
View File

@@ -1,7 +1,7 @@
const { isObject } = require("lodash"); const { isObject } = require("lodash");
const validateCanvasInputMiddleware = (req, res, next) => { const validateCanvasInputMiddleware = (req, res, next) => {
const { values, keys, override } = req.body; const { values, keys, override, w, h } = req.body;
if (!Array.isArray(values) || !Array.isArray(keys)) { if (!Array.isArray(values) || !Array.isArray(keys)) {
return res.status(400).send("Invalid input: 'values' and 'keys' must be arrays."); return res.status(400).send("Invalid input: 'values' and 'keys' must be arrays.");
@@ -19,6 +19,13 @@ const validateCanvasInputMiddleware = (req, res, next) => {
return res.status(400).send("Override must be an object"); return res.status(400).send("Override must be an object");
} }
if (w && (!Number.isFinite(w) || w <= 0)) {
return res.status(400).send("Width must be a positive number");
}
if (h && (!Number.isFinite(h) || h <= 0)) {
return res.status(400).send("Height must be a positive number");
}
next(); // Proceed to the next middleware or route handler next(); // Proceed to the next middleware or route handler
}; };