routes/login.js - Code Metrics - slistrom/me-express-back - Measure and Improve Code Quality continuously with Scrutinizer

routes/login.js A
last analyzed 2020-10-09 08:46 UTC

↳ Parent: Project

Complexity

Total Complexity	10
Complexity/F	2

Size

Lines of Code	103
Function Count	5

Duplication

Duplicated Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
wmc	10
eloc	64
mnd	5
bc	5
fnc	5
dl	0
loc	103
rs	10
bpm	1
cpm	2
noi	4
c	0
b	0
f	0

1 Function

Rating	Name	Duplication	Size	Complexity
C	login.js ➔ login	0	67	10

var express = require('express');
var router = express.Router();
// const config = require('../config');
const sqlite3 = require('sqlite3').verbose();

// const db = new sqlite3.Database('./db/texts.sqlite');
const db = require("../db/database.js");
const bcrypt = require('bcryptjs');
const jwt = require('jsonwebtoken');

let config;

try {
    config = require('../config.js');
} catch (error) {
    console.error(error);
}

const jwtSecret = process.env.JWT_SECRET || config.secret;

router.get('/', function(req, res, next) {

    const data = {
        data: {
            msg:  "Login a user"
        }
    };

    res.json(data);
});

router.post("/", (req, res) => {
    login(res, req.body);

});

function login(res, body) {
    const email = body.email;
    const password = body.password;

    if (!email || !password) {
        return res.status(401).json({
            errors: {
                status: 401,
                source: "/login",
                title: "Email or password missing",
                detail: "Email or password missing in request"
            }
        });
    }

    db.get("SELECT * FROM users WHERE email = ?",
        email,
        (err, rows) => {
        if (rows === undefined) {
            return res.status(401).json({
                errors: {
                    status: 401,
                    source: "/login",
                    title: "User not found",
                    detail: "User with provided email not found."
                }
            });
        }
        const user = rows;

        bcrypt.compare(password, user.password, (err, result) => {
            if (err) {
                return res.status(500).json({
                    errors: {
                        status: 500,
                        source: "/login",
                        title: "bcrypt error",
                        detail: "bcrypt error"
                    }
                });
            }

            if (result) {
                let payload = { email: user.email };
                let jwtToken = jwt.sign(payload, jwtSecret, { expiresIn: '24h' });

                return res.json({
                    data: {
                        type: "success",
                        message: "User logged in",
                        user: payload,
                        token: jwtToken
                    }
                });
            }

            return res.status(401).json({
                errors: {
                    status: 401,
                    source: "/login",
                    title: "Wrong password",
                    detail: "Password is incorrect."
                }
            });
        });

    });

}

module.exports = router;


1			var express = require('express');
2			var router = express.Router();
3			// const config = require('../config');
4			const sqlite3 = require('sqlite3').verbose();
			0 ignored issues – show Unused Code introduced 2020-09-28 10:17 UTC by Report Bug Copy Issue Report The constant `sqlite3` seems to be never used. Consider removing it. Loading history...
5			// const db = new sqlite3.Database('./db/texts.sqlite');
6			const db = require("../db/database.js");
7			const bcrypt = require('bcryptjs');
8			const jwt = require('jsonwebtoken');
9
10			let config;
11
12			try {
13			config = require('../config.js');
14			} catch (error) {
15			console.error(error);
16			}
17
18			const jwtSecret = process.env.JWT_SECRET \|\| config.secret;
19
20			router.get('/', function(req, res, next) {
			0 ignored issues – show Unused Code introduced 2020-09-28 10:17 UTC by Report Bug Copy Issue Report The parameter `next` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
21			const data = {
22			data: {
23			msg: "Login a user"
24			}
25			};
26
27			res.json(data);
28			});
29
30			router.post("/", (req, res) => {
31			login(res, req.body);
32
33			});
34
35			function login(res, body) {
36			const email = body.email;
37			const password = body.password;
38
39			if (!email \|\| !password) {
40			return res.status(401).json({
41			errors: {
42			status: 401,
43			source: "/login",
44			title: "Email or password missing",
45			detail: "Email or password missing in request"
46			}
47			});
48			}
49
50			db.get("SELECT * FROM users WHERE email = ?",
51			email,
52			(err, rows) => {
53			if (rows === undefined) {
54			return res.status(401).json({
55			errors: {
56			status: 401,
57			source: "/login",
58			title: "User not found",
59			detail: "User with provided email not found."
60			}
61			});
62			}
63			const user = rows;
64
65			bcrypt.compare(password, user.password, (err, result) => {
66			if (err) {
67			return res.status(500).json({
68			errors: {
69			status: 500,
70			source: "/login",
71			title: "bcrypt error",
72			detail: "bcrypt error"
73			}
74			});
75			}
76
77			if (result) {
78			let payload = { email: user.email };
79			let jwtToken = jwt.sign(payload, jwtSecret, { expiresIn: '24h' });
80
81			return res.json({
82			data: {
83			type: "success",
84			message: "User logged in",
85			user: payload,
86			token: jwtToken
87			}
88			});
89			}
90
91			return res.status(401).json({
92			errors: {
93			status: 401,
94			source: "/login",
95			title: "Wrong password",
96			detail: "Password is incorrect."
97			}
98			});
99			});
			0 ignored issues – show Best Practice introduced 2020-09-28 10:17 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
100			});
			0 ignored issues – show Best Practice introduced 2020-09-28 10:17 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
101			}
102
103			module.exports = router;
104

slistrom / me-express-back

routes/login.js A last analyzed 2020-10-09 08:46 UTC

Complexity

Size

Duplication

Importance

1 Function

Duplication Side-by-Side

Filter issues like

routes/login.js A
last analyzed 2020-10-09 08:46 UTC