fix jwt auth

app.js

@@ -43,11 +43,8 @@ app.use((req, res, next) => {
 app.use('/currency', currencyConverterRoutes);
 app.use('/auth', authRoutes);
 
-app.get('/', (req, res) => {
-	res.send('Server is running');
-});
-
 app.use((error, req, res, next) => {
+	console.log(error.statusCode);
 	const status = error.statusCode || 500;
 	const message = error.message;
 	const data = error.data;

controllers/auth.controller.js

@@ -12,9 +12,13 @@ exports.signup = (req, res, next) => {
 				firstName: req.body.firstName,
 				lastName: req.body.lastName,
 				email: req.body.email,
-				hash_password: hashedPw,
-				image: req.file.path
+				hash_password: hashedPw
 			});
+
+			if (req.file) {
+				user.image = req.file.path;
+			}
+
 			return user.save();
 		})
 		.then((user) => {

controllers/curr-converter.controller.js

@@ -16,7 +16,7 @@ exports.getConversion = (req, res, next) => {
 
 	rp(requestOptions)
 		.then((response) => {
-			console.log('API call response:', response);
+			// console.log('API call response:', response);
 			res.json(response);
 		})
 		.catch((err) => {

middleware/auth/is-auth.js

@@ -12,10 +12,10 @@ exports.isLoggedIn = (req, res, next) => {
 	const token = authHeader.split(' ')[1];
 	let decodedToken;
 	try {
-		decodedToken = jwt.verify(token, config.js);
+		decodedToken = jwt.verify(token, config.secret);
 	} catch (err) {
-		err.statusCode = 500;
-		throw error;
+		err.statusCode = 403;
+		throw err;
 	}
 	req.userId = decodedToken.userId;
 	next();

models/user.model.js

@@ -6,7 +6,7 @@ let UserSchema = new Schema({
 	lastName: { type: String, required: true },
 	email: { type: String, required: true },
 	hash_password: { type: String, required: true },
-	image: { type: String, required: true }
+	image: { type: String, required: false }
 });
 
 // Export the model

routes/curr-converter.route.js

@@ -1,7 +1,8 @@
 const express = require('express');
 const router = express.Router();
 const currencyController = require('../controllers/curr-converter.controller');
+const { isLoggedIn } = require('../middleware/auth/is-auth');
 
-router.get('/', currencyController.getConversion);
+router.get('/', isLoggedIn, currencyController.getConversion);
 
 module.exports = router;