Skip to content

Commit 51f0587

Browse files
committed
added userAuthorization middleware
1 parent 7dcd885 commit 51f0587

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

controllers/users.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -803,10 +803,6 @@ const rejectProfileDiff = async (req, res) => {
803803
};
804804

805805
const addUserIntro = async (req, res) => {
806-
if (req.path !== "/self/intro" && req.params.userId !== req.userData.id) {
807-
return res.boom.forbidden("Forbidden access");
808-
}
809-
810806
try {
811807
const rawData = req.body;
812808
const joinData = await userQuery.getJoinData(req.userData.id);

routes/users.js

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ const ROLES = require("../constants/roles");
1313
const { Services } = require("../constants/bot");
1414
const authenticateProfile = require("../middlewares/authenticateProfile");
1515
const { devFlagMiddleware } = require("../middlewares/devFlag");
16+
const { userAuthorization } = require("../middlewares/userAuthorization");
1617

1718
router.post("/", authorizeAndAuthenticate([ROLES.SUPERUSER], [Services.CRON_JOB_HANDLER]), users.markUnverified);
1819
router.post("/update-in-discord", authenticate, authorizeRoles([SUPERUSER]), users.setInDiscordScript);
@@ -37,7 +38,14 @@ router.patch(
3738
router.get("/:username", users.getUser);
3839
router.get("/:userId/intro", authenticate, authorizeRoles([SUPERUSER]), users.getUserIntro);
3940
router.put("/self/intro", authenticate, userValidator.validateJoinData, users.addUserIntro); // This route is being deprecated soon, please use alternate available route `/users/:userId/intro`.
40-
router.put("/:userId/intro", devFlagMiddleware, authenticate, userValidator.validateJoinData, users.addUserIntro);
41+
router.put(
42+
"/:userId/intro",
43+
devFlagMiddleware,
44+
authenticate,
45+
userValidator.validateJoinData,
46+
userAuthorization,
47+
users.addUserIntro
48+
);
4149
router.get("/:id/skills", users.getUserSkills);
4250
router.get("/:id/badges", getUserBadges);
4351
router.patch(

0 commit comments

Comments
 (0)