refactor(ci-visibility): simplify DI logs writer (#7373)

watson · web-flow · commit a2ff6675ccb1 · 2026-01-29T12:00:15.000Z
Remove unused tags parameter from agent proxy and add default
timeout value to DI logs writer constructor. This consolidates
the default timeout logic and removes unnecessary parameter
passing.
diff --git a/packages/dd-trace/src/ci-visibility/exporters/agent-proxy/index.js b/packages/dd-trace/src/ci-visibility/exporters/agent-proxy/index.js
@@ -73,7 +73,6 @@ class AgentProxyCiVisibilityExporter extends CiVisibilityExporter {
             const DynamicInstrumentationLogsWriter = require('../agentless/di-logs-writer')
             this._logsWriter = new DynamicInstrumentationLogsWriter({
               url: this._url,
-              tags,
               isAgentProxy: true
             })
             this._canForwardLogs = true
diff --git a/packages/dd-trace/src/ci-visibility/exporters/agentless/di-logs-writer.js b/packages/dd-trace/src/ci-visibility/exporters/agentless/di-logs-writer.js
@@ -11,7 +11,8 @@ const BaseWriter = require('../../../exporters/common/writer')
 // It is used to encode and send logs to both the logs intake directly and the
 // `/debugger/v1/input` endpoint in the agent, which is a proxy to the logs intake.
 class DynamicInstrumentationLogsWriter extends BaseWriter {
-  constructor ({ url, timeout, isAgentProxy = false }) {
+  // TODO: what's a good value for timeout for the logs intake?
+  constructor ({ url, timeout = 15_000, isAgentProxy = false }) {
     super(...arguments)
     this._url = url
     this._encoder = new JSONEncoder()
@@ -27,8 +28,7 @@ class DynamicInstrumentationLogsWriter extends BaseWriter {
         'dd-api-key': getValueFromEnvSources('DD_API_KEY'),
         'Content-Type': 'application/json'
       },
-      // TODO: what's a good value for timeout for the logs intake?
-      timeout: this.timeout || 15_000,
+      timeout: this.timeout,
       url: this._url
     }
 
