Skip to content

Commit 238cd17

Browse files
committed
Revert "added one userauthorization middleware and test cases fixes"
This reverts commit 7dcd885.
1 parent 51f0587 commit 238cd17

File tree

4 files changed

+1
-108
lines changed

4 files changed

+1
-108
lines changed

controllers/stocks.js

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -48,31 +48,6 @@ const getSelfStocks = async (req, res) => {
4848
try {
4949
const { id: userId } = req.userData;
5050
const userStocks = await stocks.fetchUserStocks(userId);
51-
52-
res.set(
53-
"X-Deprecation-Warning",
54-
"WARNING: This endpoint is being deprecated and will be removed in the future. Please use `/stocks/:userId` route to get the user stocks details."
55-
);
56-
return res.json({
57-
message: userStocks.length > 0 ? "User stocks returned successfully!" : "No stocks found",
58-
userStocks,
59-
});
60-
} catch (err) {
61-
logger.error(`Error while getting user stocks ${err}`);
62-
return res.boom.badImplementation(INTERNAL_SERVER_ERROR);
63-
}
64-
};
65-
66-
/**
67-
* Fetches all the stocks of the authenticated user
68-
*
69-
* @param req {Object} - Express request object
70-
* @param res {Object} - Express response object
71-
*/
72-
const getUserStocks = async (req, res) => {
73-
try {
74-
const userStocks = await stocks.fetchUserStocks(req.params.userId);
75-
7651
return res.json({
7752
message: userStocks.length > 0 ? "User stocks returned successfully!" : "No stocks found",
7853
userStocks,

middlewares/userAuthorization.ts

Lines changed: 0 additions & 9 deletions
This file was deleted.

routes/stocks.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,9 @@ const authorizeRoles = require("../middlewares/authorizeRoles");
55
const { addNewStock, fetchStocks, getSelfStocks } = require("../controllers/stocks");
66
const { createStock } = require("../middlewares/validators/stocks");
77
const { SUPERUSER } = require("../constants/roles");
8-
const { devFlagMiddleware } = require("../middlewares/devFlag");
9-
const { userAuthorization } = require("../middlewares/userAuthorization");
108

119
router.get("/", fetchStocks);
1210
router.post("/", authenticate, authorizeRoles([SUPERUSER]), createStock, addNewStock);
13-
router.get("/user/self", authenticate, getSelfStocks); // this route will soon be deprecated, please use `/stocks/:userId` route.
14-
router.get("/:userId", devFlagMiddleware, authenticate, userAuthorization, getUserStocks);
11+
router.get("/user/self", authenticate, getSelfStocks);
1512

1613
module.exports = router;

test/unit/middlewares/userAuthorization.test.ts

Lines changed: 0 additions & 70 deletions
This file was deleted.

0 commit comments

Comments
 (0)