Remove includeTraceContext configuration option

- the option is no longer used: open-telemetry/opentelemetry-js#3817
pinojs · Sep 12, 2023 · 194daca · 194daca
1 parent 7652365
commit 194daca
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/otlp-logger.js b/otlp-logger.js
@@ -24,7 +24,6 @@ const DEFAULT_MESSAGE_KEY = 'msg'
  * @property {string} loggerName
  * @property {string} serviceVersion
  * @property {Object} [resourceAttributes={}]
- * @property {boolean} includeTraceContext
  * @property {import('@opentelemetry/sdk-logs').LogRecordExporter} [logRecordExporter]
  * @property {boolean} [useBatchProcessor=true]
  * @property {string} [messageKey="msg"]
@@ -55,9 +54,7 @@ function getOtlpLogger (opts) {
 
   logs.setGlobalLoggerProvider(loggerProvider)
 
-  const logger = logs.getLogger(opts.loggerName, opts.serviceVersion, {
-    includeTraceContext: opts.includeTraceContext ?? true
-  })
+  const logger = logs.getLogger(opts.loggerName, opts.serviceVersion)
 
   const mapperOptions = {
     messageKey: opts.messageKey || DEFAULT_MESSAGE_KEY

diff --git a/test/otlp-logger.test.js b/test/otlp-logger.test.js
@@ -21,7 +21,6 @@ test('otlp logger logs a record in log exporter and maps all log levels correctl
       foo: 'bar'
     },
     serviceVersion: '1.0.0',
-    includeTraceContext: true,
     messageKey: 'msg',
     logRecordExporter: exporter,
     useBatchProcessor: false