From 28b0ddfb0e6d134f4f53c5594320ed52cc98fa44 Mon Sep 17 00:00:00 2001 From: Leah Schumann Date: Wed, 6 Mar 2024 07:36:16 -0500 Subject: [PATCH 1/5] Added Test environment to environments which display the log entries in the browser console. --- src/helpers/logger.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/helpers/logger.js b/src/helpers/logger.js index 7e2a3d7c1..0a172fecb 100644 --- a/src/helpers/logger.js +++ b/src/helpers/logger.js @@ -42,7 +42,8 @@ export class Logger { // If running locally or in the Dev environment show the log entries in the console. if ( applicationConfig.CURRENT_ENVIRONMENT === "Localhost" || - applicationConfig.CURRENT_ENVIRONMENT === "Dev" + applicationConfig.CURRENT_ENVIRONMENT === "Dev" || + applicationConfig.CURRENT_ENVIRONMENT === "Test" ) { console.log(logEntry); } From 8c4984a9e86623bd3452541d395572c36d6de602 Mon Sep 17 00:00:00 2001 From: Leah Schumann Date: Fri, 8 Mar 2024 08:01:40 -0500 Subject: [PATCH 2/5] Updated logger to fix env variable error and to add color coded log entries for the browser console --- src/constants/application-config.js | 1 - src/helpers/logger.js | 19 +++++++++++++++++-- 2 files changed, 17 insertions(+), 3 deletions(-) diff --git a/src/constants/application-config.js b/src/constants/application-config.js index 6c4ecd387..b8a59a129 100644 --- a/src/constants/application-config.js +++ b/src/constants/application-config.js @@ -28,7 +28,6 @@ const applicationConfig = { YAHOO_CALENDAR: "https://calendar.yahoo.com/?v=60", OUTLOOK_CALENDAR: "https://outlook.office.com/calendar/deeplink/compose?path=/calendar/action/compose&rru=addevent", - FRONTEND_LOGGER_URL: process.env.VUE_APP_CONSUMER_CF_DISTRO + "/analytics/api/v1/logging", }; export { applicationConfig }; diff --git a/src/helpers/logger.js b/src/helpers/logger.js index 0a172fecb..48a098340 100644 --- a/src/helpers/logger.js +++ b/src/helpers/logger.js @@ -45,10 +45,25 @@ export class Logger { applicationConfig.CURRENT_ENVIRONMENT === "Dev" || applicationConfig.CURRENT_ENVIRONMENT === "Test" ) { - console.log(logEntry); + switch (endpoint) { + case "log-information": + console.info(logEntry); + break; + case "log-warning": + console.warn(logEntry); + break; + case "log-error": + console.error(logEntry); + break; + case "log-critical": + console.error(logEntry); + break; + default: + console.log(logEntry); + } } - const url = applicationConfig.FRONTEND_LOGGER_URL; + const url = applicationConfig.CONSUMER_CF_DISTRO + applicationConfig.FRONTEND_LOGGER_URL; const headers = { [headerKeys.EXPERIMENT]: JSON.stringify(store.getters.experimentSettings), }; From 537158eff508b189cb2df1a20c596634932f8989 Mon Sep 17 00:00:00 2001 From: Leah Schumann Date: Fri, 8 Mar 2024 08:04:46 -0500 Subject: [PATCH 3/5] Changed app setting name for logging path --- src/constants/application-config.js | 1 + src/helpers/logger.js | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/constants/application-config.js b/src/constants/application-config.js index b8a59a129..25d24d5c8 100644 --- a/src/constants/application-config.js +++ b/src/constants/application-config.js @@ -28,6 +28,7 @@ const applicationConfig = { YAHOO_CALENDAR: "https://calendar.yahoo.com/?v=60", OUTLOOK_CALENDAR: "https://outlook.office.com/calendar/deeplink/compose?path=/calendar/action/compose&rru=addevent", + FRONTEND_LOGGER_PATH: "/analytics/api/v1/logging", }; export { applicationConfig }; diff --git a/src/helpers/logger.js b/src/helpers/logger.js index 48a098340..c8ff400ec 100644 --- a/src/helpers/logger.js +++ b/src/helpers/logger.js @@ -63,7 +63,7 @@ export class Logger { } } - const url = applicationConfig.CONSUMER_CF_DISTRO + applicationConfig.FRONTEND_LOGGER_URL; + const url = applicationConfig.CONSUMER_CF_DISTRO + applicationConfig.FRONTEND_LOGGER_PATH; const headers = { [headerKeys.EXPERIMENT]: JSON.stringify(store.getters.experimentSettings), }; From 45a2b5d022801666a5e60a07c47fda0e7bc709a5 Mon Sep 17 00:00:00 2001 From: Leah Schumann Date: Fri, 8 Mar 2024 08:10:48 -0500 Subject: [PATCH 4/5] Prettified --- src/helpers/logger.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/helpers/logger.js b/src/helpers/logger.js index c8ff400ec..4c2e6d131 100644 --- a/src/helpers/logger.js +++ b/src/helpers/logger.js @@ -59,11 +59,12 @@ export class Logger { console.error(logEntry); break; default: - console.log(logEntry); + console.log(logEntry); } } - const url = applicationConfig.CONSUMER_CF_DISTRO + applicationConfig.FRONTEND_LOGGER_PATH; + const url = + applicationConfig.CONSUMER_CF_DISTRO + applicationConfig.FRONTEND_LOGGER_PATH; const headers = { [headerKeys.EXPERIMENT]: JSON.stringify(store.getters.experimentSettings), }; From c9702aa569968dc7ddf9298a967916ef7c16d343 Mon Sep 17 00:00:00 2001 From: Leah Schumann Date: Fri, 8 Mar 2024 08:48:02 -0500 Subject: [PATCH 5/5] Changed check for environment to be 'SysTest' instead of 'Test --- src/helpers/logger.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/helpers/logger.js b/src/helpers/logger.js index 4c2e6d131..53c2cb808 100644 --- a/src/helpers/logger.js +++ b/src/helpers/logger.js @@ -43,19 +43,19 @@ export class Logger { if ( applicationConfig.CURRENT_ENVIRONMENT === "Localhost" || applicationConfig.CURRENT_ENVIRONMENT === "Dev" || - applicationConfig.CURRENT_ENVIRONMENT === "Test" + applicationConfig.CURRENT_ENVIRONMENT === "SysTest" ) { switch (endpoint) { - case "log-information": + case loggingEndpointMethods.LOG_INFORMATION: console.info(logEntry); break; - case "log-warning": + case loggingEndpointMethods.LOG_WARNING: console.warn(logEntry); break; - case "log-error": + case loggingEndpointMethods.LOG_ERROR: console.error(logEntry); break; - case "log-critical": + case loggingEndpointMethods.LOG_CRITICAL: console.error(logEntry); break; default: