Add env variables to control logging level (#1213)

microsoft · Sep 25, 2023 · 1721de1 · 1721de1
1 parent aaafbfd
commit 1721de1
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/Library/JsonConfig.ts b/Library/JsonConfig.ts
@@ -142,7 +142,7 @@ export class JsonConfig implements IJsonConfig {
                 jsonString = fs.readFileSync(tempDir, "utf8");
             }
             catch (err) {
-                Logging.info("Failed to read JSON config file: ", err);
+                Logging.warn("Failed to read JSON config file: ", err);
             }
         }
 
@@ -238,7 +238,7 @@ export class JsonConfig implements IJsonConfig {
             this.quickPulseHost = jsonConfig.quickPulseHost;
         }
         catch (err) {
-            Logging.info("Invalid JSON config file: ", err);
+            Logging.warn("Invalid JSON config file: ", err);
         }
     }
 }
diff --git a/Library/Logging.ts b/Library/Logging.ts
@@ -1,8 +1,12 @@
 import InternalAzureLogger = require("./InternalAzureLogger");
 
+
+const ENV_enableDebugLogging = "APPLICATION_INSIGHTS_ENABLE_DEBUG_LOGS";
+const ENV_disableWarningLogging = "APPLICATION_INSIGHTS_DISABLE_WARNING_LOGS";
+
 class Logging {
-    public static enableDebug = false;
-    public static disableWarnings = false;
+    public static enableDebug = (process.env[ENV_enableDebugLogging]) ? true : false;
+    public static disableWarnings = (process.env[ENV_disableWarningLogging]) ? true : false;
 
     private static TAG = "ApplicationInsights:";
 

diff --git a/README.md b/README.md
@@ -549,7 +549,7 @@ appInsights.setup("<YOUR_CONNECTION_STRING>")
     .start();
 ```
 
-Logs could be put into local file using `APPLICATIONINSIGHTS_LOG_DESTINATION` environment variable, supported values are `file` and `file+console`, a file named `applicationinsights.log` will be generated on tmp folder by default, including all logs,  `/tmp` for *nix and `USERDIR/AppData/Local/Temp` for Windows. Log directory could be configured using `APPLICATIONINSIGHTS_LOGDIR` environment variable.
+Debug Logs could be enabled as well using APPLICATION_INSIGHTS_ENABLE_DEBUG_LOGS environment variable, and APPLICATION_INSIGHTS_DISABLE_WARNING_LOGS environment variable to disable warnings. Logs could be put into local file using `APPLICATIONINSIGHTS_LOG_DESTINATION` environment variable, supported values are `file` and `file+console`, a file named `applicationinsights.log` will be generated on tmp folder by default, including all logs,  `/tmp` for *nix and `USERDIR/AppData/Local/Temp` for Windows. Log directory could be configured using `APPLICATIONINSIGHTS_LOGDIR` environment variable.
 
 ```javascript
 process.env.APPLICATIONINSIGHTS_LOG_DESTINATION = "file";