Fix: undefined limit causes crash in Sequelize

This commit is contained in:
Nicholas Wallace 2024-10-11 09:32:12 -07:00
parent d258b42e01
commit 7b8709f029

View file

@ -1246,13 +1246,11 @@ class LibraryController {
// Ensure pagination query params are positive integers // Ensure pagination query params are positive integers
for (const queryKey of ['limit', 'page']) { for (const queryKey of ['limit', 'page']) {
if (req.query[queryKey] !== undefined) { req.query[queryKey] = req.query[queryKey] && !isNaN(req.query[queryKey]) ? Number(req.query[queryKey]) : 0
req.query[queryKey] = !isNaN(req.query[queryKey]) ? Number(req.query[queryKey]) : 0
if (!Number.isInteger(req.query[queryKey]) || req.query[queryKey] < 0) { if (!Number.isInteger(req.query[queryKey]) || req.query[queryKey] < 0) {
return res.status(400).send(`Invalid request. ${queryKey} must be a positive integer`) return res.status(400).send(`Invalid request. ${queryKey} must be a positive integer`)
} }
} }
}
next() next()
} }