All tests passing

.gitignore

@@ -25,3 +25,4 @@ tmp
 package-lock.json
 yarn.lock
 test
+logs/

api/app.js

@@ -1,8 +1,23 @@
 const express = require(`express`);
+const pino = require(`pino`);
+const pinoHttp = require("pino-http");
 const app = express();
 const passport = require(`passport`);
 const { users } = require(`./controllers`);
 
+/**
+ * logging
+ */
+let logger = pino();
+if (process.env.name === `test`) {
+	logger = pino(pino.destination({ dest: `../logs/api.json`, sync: false }));
+}
+app.use(
+	pinoHttp({
+		logger: logger,
+	})
+);
+
 /**
  * Configure Passport
  */
@@ -80,9 +95,7 @@ app.use(function (err, req, res, next) {
 	if (res.headersSent) {
 		return next(err);
 	}
-	if (err.code && err.code === 500) {
-		console.log(err);
-	}
+	req.log.error(err);
 	res.status(err.code || 500);
 	res.json({
 		error: err.message || `Internal Server Error - "${err.message}"`,

api/controllers/users/index.js

@@ -38,7 +38,7 @@ const register = async (req, res) => {
  * @param {*} res
  */
 const login = async (req, res) => {
-	validation.validateOrThrow(schemas.login, res.body);
+	validation.validateOrThrow(schemas.login, req.body);
 
 	let user = await users.getByEmail(req.body.email);
 

api/models/users/index.js

@@ -63,8 +63,8 @@ const getById = async (id) => {
  * @param {*} user
  */
 const convertToPublicFormat = (user = {}) => {
-	user.email = user.hk || null;
-	user.id = user.sk2 || null;
+	user.email = user.email || user.hk || null;
+	user.id = user.id || user.sk2 || null;
 	if (user.hk) {
 		delete user.hk;
 	}

api/package.json

@@ -11,6 +11,7 @@
 		"jsonwebtoken": "^8.5.1",
 		"passport": "^0.4.1",
 		"passport-jwt": "^4.0.0",
+		"pino-http": "^5.3.0",
 		"shortid": "^2.2.15"
 	},
 	"devDependencies": {