Skip to content

Commit a10d538

Browse files
committed
🔒️Fix security issue
1 parent f7381b4 commit a10d538

File tree

3 files changed

+13
-4
lines changed

3 files changed

+13
-4
lines changed

routes/api/lineMaps.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,10 @@ module.exports.getLineMaps = getLineMaps;
4848

4949
adminRouter.post('/', function (req, res) {
5050
const map = req.body;
51-
52-
// logger.debug(map)
51+
if (typeof(map) != "object") {
52+
res.status(400).send("Bad request");
53+
return;
54+
}
5355

5456
const tiles = [];
5557
for (const i in map.tiles) {

routes/api/mazeMaps.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,10 @@ module.exports.getMazeMaps = getMazeMaps;
4545

4646
adminRouter.post('/', function (req, res) {
4747
const map = req.body;
48-
49-
// logger.debug(map)
48+
if (typeof(map) != "object") {
49+
res.status(400).send("Bad request");
50+
return;
51+
}
5052

5153
const cells = [];
5254
for (const i in map.cells) {

routes/api/teams.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ const { ACCESSLEVELS } = require('../../models/user');
1515
const logger = require('../../config/logger').mainLogger;
1616
const competitiondb = require('../../models/competition');
1717
const auth = require('../../helper/authLevels');
18+
const { type } = require('os');
1819

1920
const { LEAGUES_JSON } = competitiondb;
2021

@@ -358,6 +359,10 @@ adminRouter.post('/', function (req, res) {
358359

359360
adminRouter.post('/bulk', function (req, res) {
360361
const teams = req.body;
362+
if (typeof(teams) != "object") {
363+
res.status(400).send("Bad request");
364+
return;
365+
}
361366

362367
competitiondb.competition
363368
.findOne({

0 commit comments

Comments
 (0)