rainbownapkin/canopy
1
0
Fork 0
Code Issues 5 Pull requests Projects Releases Packages Wiki Activity Actions

Fixed vulnerability in changeRankController.js

Browse source
This commit is contained in:
rainbow napkin 2024-11-24 05:46:15 -05:00
parent 33026a1265
commit 9d401ae6a8
2 changed files with 16 additions and 23 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/controllers/api/admin/changeRankController.js
View file

@ -25,18 +25,31 @@ const userModel = require('../../../schemas/userSchema');
//api change rank functions
module.exports.post = async function(req, res){
try{
//Get validation results
const validResult = validationResult(req);
//If we don't have any validation errors
if(validResult.isEmpty()){
//get validated/sanatized data and use it to find our user from the Database.
const data = matchedData(req);
const user = await userModel.findOne({user: data.user});
if(user == null){
//If the user is null, scream and shout
res.status(400);
res.send({errors:[{type: "Bad Query", msg: "User not found.", date: new Date()}]});
}else if(user.user == req.session.user.user){
//If some smart-ass is trying self-privelege escalation
res.status(401);
return res.send({errors:[{type: "Unauthorized", msg: "No, you can't change your own rank. Fuck off.", date: new Date()}]});
}else if(permissionModel.rankToNum(data.rank) >= permissionModel.rankToNum(req.session.user.rank)){
//If the user is below the new rank of the user they're setting, scream and shout
res.status(401);
return res.send({errors:[{type: "Unauthorized", msg: "New rank must be below that of the user changing it.", date: new Date()}]});
}else if(permissionModel.rankToNum(user.rank) >= permissionModel.rankToNum(req.session.user.rank)){
//If the user is below the original rank of the user they're setting, scream and shout
res.status(401);
return res.send({errors:[{type: "Unauthorized", msg: "You cannot promote/demote peer/outranking users.", date: new Date()}]});
}
user.rank = data.rank;