diff --git a/packages/angular/src/sdk.ts b/packages/angular/src/sdk.ts index 99dbb0354e82..c6cf3b17fcd0 100755 --- a/packages/angular/src/sdk.ts +++ b/packages/angular/src/sdk.ts @@ -32,7 +32,7 @@ export function getDefaultIntegrations(_options: BrowserOptions = {}): Integrati // - https://github.com/getsentry/sentry-javascript/issues/5417#issuecomment-1453407097 // - https://github.com/getsentry/sentry-javascript/issues/2744 return [ - // TODO(v10): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` + // TODO(v11): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` // eslint-disable-next-line deprecation/deprecation inboundFiltersIntegration(), functionToStringIntegration(), diff --git a/packages/browser/src/integrations/breadcrumbs.ts b/packages/browser/src/integrations/breadcrumbs.ts index dfbfb581aeca..79022dc6e31e 100644 --- a/packages/browser/src/integrations/breadcrumbs.ts +++ b/packages/browser/src/integrations/breadcrumbs.ts @@ -73,7 +73,7 @@ const _breadcrumbsIntegration = ((options: Partial = {}) => return { name: INTEGRATION_NAME, setup(client) { - // TODO(v10): Remove this functionality and use `consoleIntegration` from @sentry/core instead. + // TODO(v11): Remove this functionality and use `consoleIntegration` from @sentry/core instead. if (_options.console) { addConsoleInstrumentationHandler(_getConsoleBreadcrumbHandler(client)); } diff --git a/packages/browser/src/sdk.ts b/packages/browser/src/sdk.ts index 2101e4e36074..7b5d67c636ae 100644 --- a/packages/browser/src/sdk.ts +++ b/packages/browser/src/sdk.ts @@ -26,7 +26,7 @@ export function getDefaultIntegrations(_options: Options): Integration[] { * `getDefaultIntegrations` but with an adjusted set of integrations. */ return [ - // TODO(v10): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` + // TODO(v11): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` // eslint-disable-next-line deprecation/deprecation inboundFiltersIntegration(), functionToStringIntegration(), diff --git a/packages/browser/src/tracing/linkedTraces.ts b/packages/browser/src/tracing/linkedTraces.ts index 9f383e72e5b8..330e193f80ef 100644 --- a/packages/browser/src/tracing/linkedTraces.ts +++ b/packages/browser/src/tracing/linkedTraces.ts @@ -193,8 +193,8 @@ export function addPreviousTraceSpanLink( // TODO: Remove this once EAP can store span links. We currently only set this attribute so that we // can obtain the previous trace information from the EAP store. Long-term, EAP will handle - // span links and then we should remove this again. Also throwing in a TODO(v10), to remind us - // to check this at v10 time :) + // span links and then we should remove this again. Also throwing in a TODO(v11), to remind us + // to check this at v11 time :) span.setAttribute( PREVIOUS_TRACE_TMP_SPAN_ATTRIBUTE, `${previousTraceSpanCtx.traceId}-${previousTraceSpanCtx.spanId}-${ diff --git a/packages/bun/src/sdk.ts b/packages/bun/src/sdk.ts index 641567504818..bf0e6f58ee90 100644 --- a/packages/bun/src/sdk.ts +++ b/packages/bun/src/sdk.ts @@ -30,7 +30,7 @@ export function getDefaultIntegrations(_options: Options): Integration[] { // We return a copy of the defaultIntegrations here to avoid mutating this return [ // Common - // TODO(v10): Replace with eventFiltersIntegration once we remove the deprecated `inboundFiltersIntegration` + // TODO(v11): Replace with eventFiltersIntegration once we remove the deprecated `inboundFiltersIntegration` // eslint-disable-next-line deprecation/deprecation inboundFiltersIntegration(), functionToStringIntegration(), diff --git a/packages/cloudflare/src/sdk.ts b/packages/cloudflare/src/sdk.ts index 58adf8f4e145..9d4fb8d749ae 100644 --- a/packages/cloudflare/src/sdk.ts +++ b/packages/cloudflare/src/sdk.ts @@ -25,13 +25,13 @@ export function getDefaultIntegrations(options: CloudflareOptions): Integration[ // The Dedupe integration should not be used in workflows because we want to // capture all step failures, even if they are the same error. ...(options.enableDedupe === false ? [] : [dedupeIntegration()]), - // TODO(v10): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` + // TODO(v11): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` // eslint-disable-next-line deprecation/deprecation inboundFiltersIntegration(), functionToStringIntegration(), linkedErrorsIntegration(), fetchIntegration(), - // TODO(v10): the `include` object should be defined directly in the integration based on `sendDefaultPii` + // TODO(v11): the `include` object should be defined directly in the integration based on `sendDefaultPii` requestDataIntegration(sendDefaultPii ? undefined : { include: { cookies: false } }), consoleIntegration(), ]; diff --git a/packages/core/src/integrations/requestdata.ts b/packages/core/src/integrations/requestdata.ts index 767d2764002f..a72fbed70d7e 100644 --- a/packages/core/src/integrations/requestdata.ts +++ b/packages/core/src/integrations/requestdata.ts @@ -21,7 +21,7 @@ type RequestDataIntegrationOptions = { include?: RequestDataIncludeOptions; }; -// TODO(v10): Change defaults based on `sendDefaultPii` +// TODO(v11): Change defaults based on `sendDefaultPii` const DEFAULT_INCLUDE: RequestDataIncludeOptions = { cookies: true, data: true, diff --git a/packages/deno/src/integrations/breadcrumbs.ts b/packages/deno/src/integrations/breadcrumbs.ts index 47a04b08fc93..1919cbbd9a67 100644 --- a/packages/deno/src/integrations/breadcrumbs.ts +++ b/packages/deno/src/integrations/breadcrumbs.ts @@ -42,7 +42,7 @@ const _breadcrumbsIntegration = ((options: Partial = {}) => return { name: INTEGRATION_NAME, setup(client) { - // TODO(v10): Remove this functionality and use `consoleIntegration` from @sentry/core instead. + // TODO(v11): Remove this functionality and use `consoleIntegration` from @sentry/core instead. if (_options.console) { addConsoleInstrumentationHandler(_getConsoleBreadcrumbHandler(client)); } diff --git a/packages/deno/src/sdk.ts b/packages/deno/src/sdk.ts index 588d417f5ed9..955417842f94 100644 --- a/packages/deno/src/sdk.ts +++ b/packages/deno/src/sdk.ts @@ -24,7 +24,7 @@ export function getDefaultIntegrations(_options: Options): Integration[] { // We return a copy of the defaultIntegrations here to avoid mutating this return [ // Common - // TODO(v10): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` + // TODO(v11): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` // eslint-disable-next-line deprecation/deprecation inboundFiltersIntegration(), functionToStringIntegration(), diff --git a/packages/node-core/src/integrations/context.ts b/packages/node-core/src/integrations/context.ts index f30361afd77b..d763ead2fd26 100644 --- a/packages/node-core/src/integrations/context.ts +++ b/packages/node-core/src/integrations/context.ts @@ -61,7 +61,7 @@ const _nodeContextIntegration = ((options: ContextOptions = {}) => { const updatedContext = _updateContext(await cachedContext); - // TODO(v10): conditional with `sendDefaultPii` here? + // TODO(v11): conditional with `sendDefaultPii` here? event.contexts = { ...event.contexts, app: { ...updatedContext.app, ...event.contexts?.app }, diff --git a/packages/node-core/src/sdk/index.ts b/packages/node-core/src/sdk/index.ts index 7d9bf6e90fd7..eb2807193b9b 100644 --- a/packages/node-core/src/sdk/index.ts +++ b/packages/node-core/src/sdk/index.ts @@ -46,7 +46,7 @@ import { maybeInitializeEsmLoader } from './esmLoader'; export function getDefaultIntegrations(): Integration[] { return [ // Common - // TODO(v10): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` + // TODO(v11): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` // eslint-disable-next-line deprecation/deprecation inboundFiltersIntegration(), functionToStringIntegration(), diff --git a/packages/node-core/src/transports/http.ts b/packages/node-core/src/transports/http.ts index ab0754eb4487..49897dfa22b1 100644 --- a/packages/node-core/src/transports/http.ts +++ b/packages/node-core/src/transports/http.ts @@ -72,7 +72,7 @@ export function makeNodeTransport(options: NodeTransportOptions): Transport { const nativeHttpModule = isHttps ? https : http; const keepAlive = options.keepAlive === undefined ? false : options.keepAlive; - // TODO(v10): Evaluate if we can set keepAlive to true. This would involve testing for memory leaks in older node + // TODO(v11): Evaluate if we can set keepAlive to true. This would involve testing for memory leaks in older node // versions(>= 8) as they had memory leaks when using it: #2555 const agent = proxy ? (new HttpsProxyAgent(proxy) as http.Agent) diff --git a/packages/react/src/tanstackrouter.ts b/packages/react/src/tanstackrouter.ts index 11677b796ffc..49c3a8984443 100644 --- a/packages/react/src/tanstackrouter.ts +++ b/packages/react/src/tanstackrouter.ts @@ -119,7 +119,7 @@ function routeMatchToParamSpanAttributes(match: VendoredTanstackRouterRouteMatch const paramAttributes: Record = {}; Object.entries(match.params).forEach(([key, value]) => { - paramAttributes[`url.path.params.${key}`] = value; // todo(v10): remove attribute which does not adhere to Sentry's semantic convention + paramAttributes[`url.path.params.${key}`] = value; // TODO(v11): remove attribute which does not adhere to Sentry's semantic convention paramAttributes[`url.path.parameter.${key}`] = value; paramAttributes[`params.${key}`] = value; // params.[key] is an alias }); diff --git a/packages/vercel-edge/src/sdk.ts b/packages/vercel-edge/src/sdk.ts index ba83d7752ed3..5c8387c9bc7a 100644 --- a/packages/vercel-edge/src/sdk.ts +++ b/packages/vercel-edge/src/sdk.ts @@ -52,14 +52,14 @@ const nodeStackParser = createStackParser(nodeStackLineParser()); export function getDefaultIntegrations(options: Options): Integration[] { return [ dedupeIntegration(), - // TODO(v10): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` + // TODO(v11): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration` // eslint-disable-next-line deprecation/deprecation inboundFiltersIntegration(), functionToStringIntegration(), linkedErrorsIntegration(), winterCGFetchIntegration(), consoleIntegration(), - // TODO(v10): integration can be included - but integration should not add IP address etc + // TODO(v11): integration can be included - but integration should not add IP address etc ...(options.sendDefaultPii ? [requestDataIntegration()] : []), ]; }