Merge pull request #971 from appsignal/otel-diag-logger

Add OpenTelemetry diag logger
appsignal · Jan 11, 2024 · fefe599 · fefe599
2 parents c49dbf3 + 6d4cd2b
commit fefe599
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 2 deletions.
diff --git a/.changesets/add-opentelemetry-diag-logger.md b/.changesets/add-opentelemetry-diag-logger.md
@@ -0,0 +1,6 @@
+---
+bump: "patch"
+type: "add"
+---
+
+When the AppSignal log level is set to "trace". Additional information from the OpenTelemetry instrumentations is now logged.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/client.ts b/src/client.ts
@@ -9,6 +9,7 @@ import { demo } from "./demo"
 import { VERSION } from "./version"
 import { setParams, setSessionData } from "./helpers"
 import { BaseLogger, Logger, LoggerFormat, LoggerLevel } from "./logger"
+import { diag, DiagConsoleLogger, DiagLogLevel } from "@opentelemetry/api"
 
 import { Instrumentation } from "@opentelemetry/instrumentation"
 import {
@@ -157,6 +158,7 @@ export class Client {
         this.#metrics = new Metrics()
         if (this.config.data.initializeOpentelemetrySdk) {
           this.#sdk = this.initOpenTelemetry()
+          this.setUpOpenTelemetryLogger()
         }
       }
     } else {
@@ -416,6 +418,17 @@ export class Client {
     return logger
   }
 
+  /**
+   * Sets up the OpenTelemetry diag logger based on our integration logger level.
+   * If our integration logger level is "silly" ("trace"), the OpenTelemetry diag debug messages
+   * are logged.
+   */
+  private setUpOpenTelemetryLogger(): void {
+    if (this.config.data["logLevel"] === "trace") {
+      diag.setLogger(new DiagConsoleLogger(), DiagLogLevel.DEBUG)
+    }
+  }
+
   /**
    * Stores the client in global object after initializing
    */