Skip to content

Commit 6796798

Browse files
committed
chore(evp-proxy): remove cross-team TODOs
1 parent 6eb5732 commit 6796798

5 files changed

Lines changed: 0 additions & 6 deletions

File tree

.github/CODEOWNERS

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -438,7 +438,6 @@
438438
/packages/dd-trace/src/exporters/common/ @DataDog/lang-platform-js
439439
/packages/dd-trace/src/exporters/common/client-library-headers.js @DataDog/lang-platform-js @DataDog/feature-flagging-and-experimentation-sdk
440440
/packages/dd-trace/src/evp_proxy/ @DataDog/lang-platform-js
441-
/packages/dd-trace/src/guardrails/ @DataDog/lang-platform-js
442441
/packages/dd-trace/src/heap_snapshots.js @DataDog/lang-platform-js
443442
/packages/dd-trace/src/histogram.js @DataDog/lang-platform-js
444443
/packages/dd-trace/src/id.js @DataDog/lang-platform-js
@@ -460,7 +459,6 @@
460459
/packages/dd-trace/test/agent/ @DataDog/lang-platform-js
461460
/packages/dd-trace/test/dd-trace.spec.js @DataDog/lang-platform-js
462461
/packages/dd-trace/test/dogstatsd.spec.js @DataDog/lang-platform-js
463-
/packages/dd-trace/test/encode/ @DataDog/lang-platform-js
464462
/packages/dd-trace/test/evp_proxy/ @DataDog/lang-platform-js
465463
/packages/dd-trace/test/esm-named-exports.spec.js @DataDog/lang-platform-js
466464
/packages/dd-trace/test/exporter.spec.js @DataDog/lang-platform-js

packages/dd-trace/src/ci-visibility/exporters/agent-proxy/index.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ const request = require('../request')
88
const { fetchAgentInfo } = require('../../../agent/info')
99
const { DEBUGGER_INPUT_V1 } = require('../../../debugger/constants')
1010

11-
// TODO: Use the shared utilities in src/evp_proxy when this product migrates its EVP version policy.
1211
const AGENT_EVP_PROXY_PATH_PREFIX = '/evp_proxy/v'
1312
const AGENT_EVP_PROXY_PATH_REGEX = /\/evp_proxy\/v(\d+)\/?/
1413

packages/dd-trace/src/ci-visibility/exporters/agentless/writer.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ class Writer extends BaseWriter {
5656
if (this._evpProxyPrefix) {
5757
options.path = `${this._evpProxyPrefix}/api/v2/citestcycle`
5858
delete options.headers['dd-api-key']
59-
// TODO: Migrate CI Visibility EVP headers and subdomains to the shared utilities in src/evp_proxy.
6059
options.headers['X-Datadog-EVP-Subdomain'] = 'citestcycle-intake'
6160
}
6261

packages/dd-trace/src/llmobs/writers/base.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,6 @@ class BaseLLMObsWriter {
234234
if (this._agentless) {
235235
options.headers['DD-API-KEY'] = this._config.DD_API_KEY || ''
236236
} else {
237-
// TODO: Migrate LLMObs EVP headers and subdomains to the shared utilities in src/evp_proxy.
238237
options.headers[EVP_SUBDOMAIN_HEADER_NAME] = this._intake
239238
}
240239

packages/dd-trace/src/llmobs/writers/util.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ const { EVP_PROXY_AGENT_BASE_PATH } = require('../constants/writers')
55
const telemetry = require('../telemetry')
66
const { fetchAgentInfo } = require('../../agent/info')
77

8-
// TODO: Use the shared utilities in src/evp_proxy when LLMObs migrates this product-specific fallback policy.
98
/**
109
* @param {import('../../config/config-base')} config
1110
* @param {(agentless: boolean) => void} setWritersAgentlessValue

0 commit comments

Comments
 (0)