fix: #8827, do not require admin:users privilege to ban users

v1.18.x
Julian Lam 4 years ago
parent 540a32cfdd
commit 891a1ea2af

@ -59,13 +59,11 @@ Users.unfollow = async (req, res) => {
};
Users.ban = async (req, res) => {
await hasAdminPrivilege(req.uid, 'users');
await api.users.ban(req, { ...req.body, uid: req.params.uid });
helpers.formatApiResponse(200, res);
};
Users.unban = async (req, res) => {
await hasAdminPrivilege(req.uid, 'users');
await api.users.unban(req, { ...req.body, uid: req.params.uid });
helpers.formatApiResponse(200, res);
};

@ -69,8 +69,6 @@ module.exports = function (privileges) {
'admin.user.loadGroups': 'admin:users',
'admin.groups.join': 'admin:users',
'admin.groups.leave': 'admin:users',
'user.banUsers': 'admin:users',
'user.unbanUsers': 'admin:users',
'admin.user.resetLockouts': 'admin:users',
'admin.user.validateEmail': 'admin:users',
'admin.user.sendValidationEmail': 'admin:users',

Loading…
Cancel
Save