fix: set default log level to INFO

As per open-telemetry#856 Signed-off-by: Naseem <naseem@transit.app>
naseemkullah · Jul 24, 2020 · 63c1640 · 63c1640
1 parent da0fac5
commit 63c1640
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/opentelemetry-core/src/common/ConsoleLogger.ts b/packages/opentelemetry-core/src/common/ConsoleLogger.ts
@@ -19,7 +19,7 @@ import { LogLevel } from './types';
 import { getEnv } from '../platform';
 
 export class ConsoleLogger implements Logger {
-  constructor(level: LogLevel = getEnv().OTEL_LOG_LEVEL || LogLevel.INFO) {
+  constructor(level: LogLevel = getEnv().OTEL_LOG_LEVEL) {
     if (level >= LogLevel.DEBUG) {
       this.debug = (...args) => {
         console.debug(...args);

diff --git a/packages/opentelemetry-core/src/utils/environment.ts b/packages/opentelemetry-core/src/utils/environment.ts
@@ -37,7 +37,7 @@ const ENVIRONMENT_NUMBERS: Partial<keyof ENVIRONMENT>[] = [
  */
 export const DEFAULT_ENVIRONMENT: Required<ENVIRONMENT> = {
   OTEL_NO_PATCH_MODULES: '',
-  OTEL_LOG_LEVEL: LogLevel.ERROR,
+  OTEL_LOG_LEVEL: LogLevel.INFO,
   OTEL_SAMPLING_PROBABILITY: 1,
 };
 
@@ -85,7 +85,7 @@ export function parseEnvironment(values: ENVIRONMENT_MAP): ENVIRONMENT {
           const logLevelString = value.toUpperCase() as LogLevelString
           logLevel = LogLevel[logLevelString];
         }
-        environment[key] = logLevel ?? LogLevel.ERROR;
+        environment[key] = logLevel || DEFAULT_ENVIRONMENT[key]
         break;
 
       default: