fix: now logs show details and the inputs that caused the error

sanitas_backend/src/handlers/CreatePatient/create-patient.mjs

@@ -67,7 +67,7 @@ export const createPatientHandler = async (event, context) => {
 			]);
 
 			if (existingPatientResult.rows.length > 0) {
-				logger.error("CUI already exists.");
+				logger.error(patientData.cui, "CUI already exists.");
 
 				const response = responseBuilder
 					.setStatusCode(409)

sanitas_backend/src/handlers/ExportData/export-data.mjs

@@ -56,7 +56,10 @@ export const handler = async (event, context) => {
 	logger.info({ jwt }, "Parsing JWT...");
 	const tokenInfo = decodeJWT(jwt);
 	if (tokenInfo.error) {
-		logger.error({ error: tokenInfo.error }, "JWT couldn't be parsed!");
+		logger.error(
+			{ err: tokenInfo.error, inputs: { jwt } },
+			"JWT couldn't be parsed!",
+		);
 		return responseBuilder
 			.setStatusCode(400)
 			.setBody({ error: "JWT couldn't be parsed" })
@@ -115,7 +118,7 @@ export const handler = async (event, context) => {
 			if (itsDoctor.error) {
 				const msg =
 					"An error occurred while trying to check if user is doctor!";
-				logger.error({ error: itsDoctor.error }, msg);
+				logger.error({ err: itsDoctor.error, inputs: { email } }, msg);
 
 				const response = responseBuilder
 					.setStatusCode(500)
@@ -146,6 +149,10 @@ export const handler = async (event, context) => {
 		});
 
 		if (transactionResult.error) {
+			logger.error(
+				{ err: transactionResult.error },
+				"An error occurred during the database transaction!",
+			);
 			throw transactionResult.error;
 		}
 
@@ -172,7 +179,7 @@ export const handler = async (event, context) => {
 		return responseBuilder.setStatusCode(200).setBody(csv).build();
 	} catch (error) {
 		logger.error(
-			{ details: error.message },
+			{ err: error, details: error.message },
 			"An error occurred while exporting data!",
 		);
 

sanitas_backend/src/handlers/GetAllergicHistory/get-allergic-history-patient.mjs

@@ -37,7 +37,10 @@ export const getAllergicHistoryHandler = async (event, context) => {
 	logger.info({ jwt }, "Parsing JWT...");
 	const tokenInfo = decodeJWT(jwt);
 	if (tokenInfo.error) {
-		logger.error({ error: tokenInfo.error }, "JWT couldn't be parsed!");
+		logger.error(
+			{ err: tokenInfo.error, inputs: { jwt } },
+			"JWT couldn't be parsed!",
+		);
 		return responseBuilder
 			.setStatusCode(400)
 			.setBody({ error: "JWT couldn't be parsed" })
@@ -50,7 +53,10 @@ export const getAllergicHistoryHandler = async (event, context) => {
 	try {
 		const patientId = Number.parseInt(event.pathParameters.id, 10);
 		if (!patientId) {
-			logger.error("Invalid ID received!");
+			logger.error(
+				{ patientId: event.pathParameters?.id },
+				"Invalid ID received!",
+			);
 			return responseBuilder
 				.setStatusCode(400)
 				.setBody({ error: "Invalid request: No valid patientId supplied!" })
@@ -69,7 +75,7 @@ export const getAllergicHistoryHandler = async (event, context) => {
 			if (itsDoctor.error) {
 				const msg =
 					"An error occurred while trying to check if the user is a doctor!";
-				logger.error(itsDoctor, msg);
+				logger.error({ err: itsDoctor.error, inputs: { email } }, msg);
 
 				const response = responseBuilder
 					.setStatusCode(500)
@@ -89,7 +95,10 @@ export const getAllergicHistoryHandler = async (event, context) => {
 				if (emailBelongs.error) {
 					const msg =
 						"An error ocurred while trying to check if the email belongs to the patient!";
-					logger.error(emailBelongs, msg);
+					logger.error(
+						{ err: emailBelongs.error, inputs: { email, patientId } },
+						msg,
+					);
 
 					const response = responseBuilder
 						.setStatusCode(500)
@@ -127,6 +136,11 @@ export const getAllergicHistoryHandler = async (event, context) => {
 		});
 
 		if (transactionResult.error) {
+			logger.error(
+				{ err: transactionResult.error },
+				"An error occurred during the database transaction!",
+			);
+
 			throw transactionResult.error;
 		}
 
@@ -152,7 +166,7 @@ export const getAllergicHistoryHandler = async (event, context) => {
 		return responseBuilder.setStatusCode(200).setBody(medicalHistory).build();
 	} catch (error) {
 		logger.error(
-			{ error },
+			{ err: error },
 			"An error occurred while fetching allergic history!",
 		);
 		return responseBuilder

sanitas_backend/src/handlers/GetConsultationPatient/get-consultation-patient.mjs

@@ -31,7 +31,10 @@ export const getMedicalConsultationHandler = async (event, context) => {
 	logger.info({ jwt }, "Parsing JWT...");
 	const tokenInfo = decodeJWT(jwt);
 	if (tokenInfo.error) {
-		logger.error({ error: tokenInfo.error }, "JWT couldn't be parsed!");
+		logger.error(
+			{ err: tokenInfo.error, inputs: { jwt } },
+			"JWT couldn't be parsed!",
+		);
 		return responseBuilder
 			.setStatusCode(400)
 			.setBody({ error: "JWT couldn't be parsed" })
@@ -42,7 +45,10 @@ export const getMedicalConsultationHandler = async (event, context) => {
 
 	const patientId = Number.parseInt(event.pathParameters?.id, 10);
 	if (Number.isNaN(patientId)) {
-		logger.error("Invalid ID received!", { id: event.pathParameters?.id });
+		logger.error(
+			{ patientId: event.pathParameters?.id },
+			"Invalid ID received!",
+		);
 		return responseBuilder
 			.setStatusCode(400)
 			.setBody({ error: "Invalid request: No valid patientId supplied!" })
@@ -63,7 +69,7 @@ export const getMedicalConsultationHandler = async (event, context) => {
 			if (itsDoctor.error) {
 				const msg =
 					"An error occurred while trying to check if the user is a doctor!";
-				logger.error(itsDoctor, msg);
+				logger.error({ err: itsDoctor.error, inputs: { email } }, msg);
 
 				const response = responseBuilder
 					.setStatusCode(500)
@@ -100,6 +106,10 @@ export const getMedicalConsultationHandler = async (event, context) => {
 		});
 
 		if (transactionResult.error) {
+			logger.error(
+				{ err: transactionResult.error },
+				"An error occurred during the database transaction!",
+			);
 			throw transactionResult.error;
 		}
 
@@ -127,8 +137,8 @@ export const getMedicalConsultationHandler = async (event, context) => {
 			.build();
 	} catch (error) {
 		logger.error(
+			{ err: error },
 			"An error occurred while fetching medical consultation!",
-			error,
 		);
 		return responseBuilder
 			.setStatusCode(500)

sanitas_backend/src/handlers/GetFamilyHistory/get-family-history-patient.mjs

@@ -38,7 +38,10 @@ export const getFamilyHistoryHandler = async (event, context) => {
 	logger.info({ jwt }, "Parsing JWT...");
 	const tokenInfo = decodeJWT(jwt);
 	if (tokenInfo.error) {
-		logger.error({ error: tokenInfo.error }, "JWT couldn't be parsed!");
+		logger.error(
+			{ err: tokenInfo.error, inputs: { jwt } },
+			"JWT couldn't be parsed!",
+		);
 		return responseBuilder
 			.setStatusCode(400)
 			.setBody({ error: "JWT couldn't be parsed" })
@@ -51,7 +54,10 @@ export const getFamilyHistoryHandler = async (event, context) => {
 	try {
 		const patientId = Number.parseInt(event.pathParameters.id, 10);
 		if (!patientId) {
-			logger.error("Invalid ID received!");
+			logger.error(
+				{ patientId: event.pathParameters?.id },
+				"Invalid ID received!",
+			);
 			return responseBuilder
 				.setStatusCode(400)
 				.setBody({ error: "Invalid request: No valid patientId supplied!" })
@@ -70,7 +76,7 @@ export const getFamilyHistoryHandler = async (event, context) => {
 			if (itsDoctor.error) {
 				const msg =
 					"An error occurred while trying to check if the user is a doctor!";
-				logger.error(itsDoctor, msg);
+				logger.error({ err: itsDoctor.error, inputs: { email } }, msg);
 				const response = responseBuilder
 					.setStatusCode(500)
 					.setBody(itsDoctor)
@@ -89,7 +95,10 @@ export const getFamilyHistoryHandler = async (event, context) => {
 				if (emailBelongs.error) {
 					const msg =
 						"An error ocurred while trying to check if the email belongs to the patient!";
-					logger.error(emailBelongs, msg);
+					logger.error(
+						{ err: emailBelongs.error, inputs: { email, patientId } },
+						msg,
+					);
 					const response = responseBuilder
 						.setStatusCode(500)
 						.setBody(emailBelongs)
@@ -125,6 +134,10 @@ export const getFamilyHistoryHandler = async (event, context) => {
 		});
 
 		if (transactionResult.error) {
+			logger.error(
+				{ err: transactionResult.error },
+				"An error occurred during the database transaction!",
+			);
 			throw transactionResult.error;
 		}
 
@@ -149,7 +162,7 @@ export const getFamilyHistoryHandler = async (event, context) => {
 		const medicalHistory = mapToAPIFamilyHistory(dbResponse.rows[0]);
 		return responseBuilder.setStatusCode(200).setBody(medicalHistory).build();
 	} catch (error) {
-		logger.error("An error occurred while fetching family history!", error);
+		logger.error(error, "An error occurred while fetching family history!");
 		return responseBuilder
 			.setStatusCode(500)
 			.setBody({

sanitas_backend/src/handlers/GetGeneralCollaboratorInfo/get-collaborator.mjs

@@ -31,7 +31,10 @@ export const getCollaboratorHandler = async (event, context) => {
 	logger.info({ jwt }, "Parsing JWT...");
 	const tokenInfo = decodeJWT(jwt);
 	if (tokenInfo.error) {
-		logger.error({ error: tokenInfo.error }, "JWT couldn't be parsed!");
+		logger.error(
+			{ err: tokenInfo.error, inputs: { jwt } },
+			"JWT couldn't be parsed!",
+		);
 		return responseBuilder
 			.setStatusCode(400)
 			.setBody({ error: "JWT couldn't be parsed" })
@@ -43,7 +46,7 @@ export const getCollaboratorHandler = async (event, context) => {
 	logger.info("Checking if received all parameters...");
 	const patientId = event.pathParameters.id;
 	if (!patientId) {
-		logger.error("No id received!");
+		logger.error({ patientId: event.pathParameters?.id }, "No id received!");
 		const response = responseBuilder
 			.setStatusCode(400)
 			.setBody({ error: "Invalid request: No id supplied!" })
@@ -66,7 +69,7 @@ export const getCollaboratorHandler = async (event, context) => {
 			if (itsDoctor.error) {
 				const msg =
 					"An error occurred while trying to check if the user is a doctor!";
-				logger.error(itsDoctor, msg);
+				logger.error({ err: itsDoctor.error, inputs: { email } }, msg);
 				const response = responseBuilder
 					.setStatusCode(500)
 					.setBody(itsDoctor)
@@ -85,7 +88,10 @@ export const getCollaboratorHandler = async (event, context) => {
 				if (emailBelongs.error) {
 					const msg =
 						"An error ocurred while trying to check if the email belongs to the patient!";
-					logger.error(emailBelongs, msg);
+					logger.error(
+						{ err: emailBelongs.error, inputs: { email, patientId } },
+						msg,
+					);
 					const response = responseBuilder
 						.setStatusCode(500)
 						.setBody(emailBelongs)
@@ -118,6 +124,10 @@ export const getCollaboratorHandler = async (event, context) => {
 		});
 
 		if (transactionResult.error) {
+			logger.error(
+				{ err: transactionResult.error },
+				"An error occurred during the database transaction!",
+			);
 			throw transactionResult.error;
 		}
 

sanitas_backend/src/handlers/GetGeneralPatientInfo/get-general-patient-info.mjs

@@ -28,7 +28,10 @@ export const handler = async (event, context) => {
 	logger.info({ jwt }, "Parsing JWT...");
 	const tokenInfo = decodeJWT(jwt);
 	if (tokenInfo.error) {
-		logger.error({ error: tokenInfo.error }, "JWT couldn't be parsed!");
+		logger.error(
+			{ err: tokenInfo.error, inputs: { jwt } },
+			"JWT couldn't be parsed!",
+		);
 		return responseBuilder
 			.setStatusCode(400)
 			.setBody({ error: "JWT couldn't be parsed" })
@@ -40,7 +43,7 @@ export const handler = async (event, context) => {
 	logger.info("Checking if received all parameters...");
 	const patientId = event.pathParameters.id;
 	if (patientId !== 0 && !patientId) {
-		logger.error("No ID received!");
+		logger.error({ patientId: event.pathParameters?.id }, "No ID received!");
 		const response = responseBuilder
 			.setStatusCode(400)
 			.setBody({
@@ -65,7 +68,7 @@ export const handler = async (event, context) => {
 			if (itsDoctor.error) {
 				const msg =
 					"An error occurred while trying to check if the user is a doctor!";
-				logger.error(itsDoctor, msg);
+				logger.error({ err: itsDoctor.error, inputs: { email } }, msg);
 				const response = responseBuilder
 					.setStatusCode(500)
 					.setBody(itsDoctor)
@@ -84,7 +87,10 @@ export const handler = async (event, context) => {
 				if (emailBelongs.error) {
 					const msg =
 						"An error ocurred while trying to check if the email belongs to the patient!";
-					logger.error(emailBelongs, msg);
+					logger.error(
+						{ err: emailBelongs, inputs: { email, patientId } },
+						msg,
+					);
 					const response = responseBuilder
 						.setStatusCode(500)
 						.setBody(emailBelongs)
@@ -117,6 +123,10 @@ export const handler = async (event, context) => {
 		});
 
 		if (transactionResult.error) {
+			logger.error(
+				{ err: transactionResult.error },
+				"An error occurred during the database transaction!",
+			);
 			throw transactionResult.error;
 		}
 
@@ -147,7 +157,7 @@ export const handler = async (event, context) => {
 		logger.info({ response }, "Responding with:");
 		return response;
 	} catch (error) {
-		logger.error({ error }, "An error has ocurred!");
+		logger.error(error, "An error has ocurred!");
 		const response = responseBuilder.setStatusCode(500).setBody(error).build();
 		return response;
 	} finally {