Node: Check debug mode for debug logs (#1046)

* check debug mode for debug logs * Bump package version * Update changelog * re-bump version * Use dynamic feature flag variable * Fix condition
microsoft · Jul 29, 2024 · a783d8a · a783d8a
1 parent 51ae910
commit a783d8a
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 6 deletions.
diff --git a/node/CHANGELOG.md b/node/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## 4.x
 
+## 4.15.0
+
+- Disabled debug logs when debug mode is not enabled - [#1046](https://github.com/microsoft/azure-pipelines-task-lib/pull/1046)
+
 ## 4.12.1
 
 - Remove deasync from task-lib - [#1038](https://github.com/microsoft/azure-pipelines-task-lib/pull/1038)

diff --git a/node/internal.ts b/node/internal.ts
@@ -344,8 +344,16 @@ export function _error(
     );
 }
 
+const debugMode = _getVariable('system.debug')?.toLowerCase() === 'true';
+const shouldCheckDebugMode = _getVariable('DistributedTask.Tasks.Node.SkipDebugLogsWhenDebugModeOff')?.toLowerCase() === 'true';
+
 export function _debug(message: string): void {
-    _command('task.debug', null, message);
+    if (
+        !shouldCheckDebugMode
+        || (shouldCheckDebugMode && debugMode)
+    ) {
+        _command('task.debug', null, message);
+    }
 }
 
 // //-----------------------------------------------------

diff --git a/node/package-lock.json b/node/package-lock.json
diff --git a/node/package.json b/node/package.json
@@ -1,6 +1,6 @@
 {
   "name": "azure-pipelines-task-lib",
-  "version": "4.14.0",
+  "version": "4.15.0",
   "description": "Azure Pipelines Task SDK",
   "main": "./task.js",
   "typings": "./task.d.ts",