Skip to content

Commit

Permalink
feat: loginRequired middleware added (foyzulkarim#35)
Browse files Browse the repository at this point in the history
  • Loading branch information
aninda052 committed Aug 30, 2023
1 parent a3f147d commit f3e35b3
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 5 deletions.
1 change: 1 addition & 0 deletions server/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@
"fluent-ffmpeg": "^2.1.2",
"helmet": "^6.0.1",
"joi": "^17.7.0",
"jsonwebtoken": "^9.0.2",
"mongodb": "^5.7.0",
"multer": "^1.4.5-lts.1",
"nodemon": "^3.0.1",
Expand Down
13 changes: 12 additions & 1 deletion server/src/middleware/auth.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,19 @@ const setCurrentUser = async (req, res, next) => {
next();
};

const loginRequired = (req, res, next) => {

if(req.user){
next();
}else{
return res.status(401).json({message: 'you must login to access this resource' });
}

};


module.exports = {
setCurrentUser
setCurrentUser,
loginRequired
}

1 change: 0 additions & 1 deletion server/src/modules/authentication/controller.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
const { loginValidate, authenticate } = require('./request');
const { generateJwtToken } = require('./utils');
const { setCookie } = require('../../utils/cookie');

const setupRoutes = (app) => {

Expand Down
7 changes: 4 additions & 3 deletions server/src/modules/models/video/controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ const { addQueueItem } = require('../../queues/queue');
const {
getVideoDurationAndResolution,
} = require('../../queues/video-processor');
const { loginRequired } = require('../../../middleware/auth');

const BASE_URL = `/api/videos`;

Expand Down Expand Up @@ -69,7 +70,7 @@ const setupRoutes = (app) => {
// .json({ status: 'error', message: validationResult.error });
// });

app.put(`${BASE_URL}/update/:id`, async (req, res) => {
app.put(`${BASE_URL}/update/:id`, loginRequired, async (req, res) => {
const validationResult = validate(req.body);
if (req.params.id && !validationResult.error) {
const result = await update({_id:req.params.id, ...validationResult.value});
Expand All @@ -83,7 +84,7 @@ const setupRoutes = (app) => {
.json({ status: 'error', message: validationResult.error });
});

app.delete(`${BASE_URL}/delete/:id`, async (req, res) => {
app.delete(`${BASE_URL}/delete/:id`, loginRequired, async (req, res) => {
console.log('DELETE', req.params.id);
if (req.params.id) {
const result = await deleteById(req.params.id);
Expand Down Expand Up @@ -139,7 +140,7 @@ const setupRoutes = (app) => {
});
};

app.post(`${BASE_URL}/upload`, uploadProcessor, async (req, res) => {
app.post(`${BASE_URL}/upload`, loginRequired, uploadProcessor, async (req, res) => {
try {
// const { videoDuration } = await getVideoDurationAndResolution(
// `./${req.file.path}`
Expand Down

0 comments on commit f3e35b3

Please sign in to comment.