From 6059f13fa147d4087872576fe05852b939323f33 Mon Sep 17 00:00:00 2001
From: Pritish Budhiraja <pritish.budhiraja@gmail.com>
Date: Mon, 28 Oct 2024 13:10:25 +0530
Subject: [PATCH] refactor: disable logging in integration environment (#753)

---
 package.json      | 25 ++++++++++++-------------
 webpack.common.js |  4 ++--
 2 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/package.json b/package.json
index 4ceafaf2..57ab68d4 100644
--- a/package.json
+++ b/package.json
@@ -16,23 +16,22 @@
     "webpack-merge": "^5.9.0"
   },
   "scripts": {
-    "build:dev": "cross-env sdkEnv=sandbox webpack --config webpack.dev.js",
-    "build:dev-integ": "cross-env sdkEnv=integ webpack --config webpack.dev.js",
-    "start": "cross-env sdkEnv=local webpack serve --config webpack.dev.js",
-    "build:prod": "cross-env sdkEnv=prod webpack --config webpack.common.js",
     "build": "webpack --config webpack.common.js",
-    "build:sandbox": "cross-env sdkEnv=sandbox webpack --config webpack.common.js",
-    "build:integ": "cross-env sdkEnv=integ webpack --config webpack.common.js",
-    "test": "cd cypress-tests && npm run cypress:run",
+    "build:integ": "cross-env sdkEnv=integ enableLogging=false webpack --config webpack.common.js",
+    "build:playground": "npm run setup:playground && npm run build",
+    "build:prod": "cross-env sdkEnv=prod enableLogging=true webpack --config webpack.common.js",
+    "build:sandbox": "cross-env sdkEnv=sandbox enableLogging=true webpack --config webpack.common.js",
+    "deploy-to-s3": "node ./scripts/pushToS3.js",
+    "postinstall": "cd Hyperswitch-React-Demo-App && npm i",
+    "prepare": "husky install",
     "re:build": "rescript",
     "re:clean": "rescript clean",
-    "re:start": "rescript -w",
     "re:format": "rescript format -all",
-    "start:playground": "npm run postinstall && cd Hyperswitch-React-Demo-App && node promptScript.js && npm run start",
-    "build:playground": "npm run postinstall && cd Hyperswitch-React-Demo-App && node promptScript.js && npm run build",
-    "prepare": "husky install",
-    "deploy-to-s3": "node ./scripts/pushToS3.js",
-    "postinstall": "cd Hyperswitch-React-Demo-App && npm i",
+    "re:start": "rescript -w",
+    "setup:playground": "npm run postinstall && cd Hyperswitch-React-Demo-App && node promptScript.js",
+    "start": "cross-env sdkEnv=local enableLogging=false webpack serve --config webpack.dev.js",
+    "start:playground": "npm run setup:playground && npm run start",
+    "test": "cd cypress-tests && npm run cypress:run",
     "test:hooks": "npx eslint src/"
   },
   "eslintConfig": {
diff --git a/webpack.common.js b/webpack.common.js
index 981f2a97..4357ab07 100644
--- a/webpack.common.js
+++ b/webpack.common.js
@@ -14,6 +14,7 @@ const getEnvVariable = (variable, defaultValue) =>
   process.env[variable] ?? defaultValue;
 
 const sdkEnv = getEnvVariable("sdkEnv", "local");
+const enableLogging = getEnvVariable("enableLogging", "false") === "true";
 const envSdkUrl = getEnvVariable("ENV_SDK_URL", "");
 const envBackendUrl = getEnvVariable("ENV_BACKEND_URL", "");
 const envLoggingUrl = getEnvVariable("ENV_LOGGING_URL", "");
@@ -60,7 +61,6 @@ const confirmEndPoint =
 const logEndpoint =
   envLoggingUrl || `https://${logDomain}.hyperswitch.io/logs/sdk`;
 
-const enableLogging = true;
 const loggingLevel = "DEBUG";
 const maxLogsPushedPerEventName = 100;
 
@@ -84,7 +84,7 @@ module.exports = (publicPath = "auto") => {
       logEndpoint: JSON.stringify(logEndpoint),
       sentryDSN: JSON.stringify(process.env.SENTRY_DSN),
       sentryScriptUrl: JSON.stringify(process.env.SENTRY_SCRIPT_URL),
-      enableLogging: JSON.stringify(enableLogging),
+      enableLogging: enableLogging,
       loggingLevel: JSON.stringify(loggingLevel),
       maxLogsPushedPerEventName: JSON.stringify(maxLogsPushedPerEventName),
     }),