vidly-api-node icon indicating copy to clipboard operation
vidly-api-node copied to clipboard

Increased auth performance

Open goto3 opened this issue 6 years ago • 1 comments

Hello, I am a starting with node.js and I found something interesting:

On auth file in routes folder I changed this on line 17:

const validPassword = await bcrypt.compare(req.body.password, user.password);
if (!validPassword) return res.status(400).send("Invalid email or password.");

image

to this:

await bcrypt.compare(req.body.password, user.password, function(err, result) {
    if (!result) return res.status(400).send("Invalid email or password.");
});

image

and decreased request processing time from 85 ms to 5 ms average.

Why is that?

Note: I am using bcrypt v3.0.6

Thank you very much

goto3 avatar Jul 17 '19 00:07 goto3

Because the request was cached. Try run different users ,and you will see.

rustemjantayev avatar Sep 09 '19 01:09 rustemjantayev