Skip to content

Commit e781ebe

Browse files
authored
feat(core): Rename inboundFiltersIntegration to eventFiltersIntegration (#15434)
Deprecate and rename the `inboundFiltersIntegration` to `eventFiltersIntegration` to improve the integration name and avoid ambiguity with the product.
1 parent 082e0e8 commit e781ebe

File tree

27 files changed

+165
-77
lines changed

27 files changed

+165
-77
lines changed

dev-packages/browser-integration-tests/suites/manual-client/browser-context/init.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import {
66
defaultStackParser,
77
functionToStringIntegration,
88
httpContextIntegration,
9-
inboundFiltersIntegration,
9+
eventFiltersIntegration,
1010
linkedErrorsIntegration,
1111
makeFetchTransport,
1212
} from '@sentry/browser';
@@ -16,7 +16,7 @@ const integrations = [
1616
functionToStringIntegration(),
1717
dedupeIntegration(),
1818
httpContextIntegration(),
19-
inboundFiltersIntegration(),
19+
eventFiltersIntegration(),
2020
linkedErrorsIntegration(),
2121
];
2222

dev-packages/browser-integration-tests/suites/manual-client/browser-context/test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ sentryTest('allows to setup a client manually & capture exceptions', async ({ ge
3838
environment: 'local',
3939
release: '0.0.1',
4040
sdk: {
41-
integrations: ['Breadcrumbs', 'FunctionToString', 'Dedupe', 'HttpContext', 'InboundFilters', 'LinkedErrors'],
41+
integrations: ['Breadcrumbs', 'FunctionToString', 'Dedupe', 'HttpContext', 'EventFilters', 'LinkedErrors'],
4242
name: 'sentry.javascript.browser',
4343
version: expect.any(String),
4444
packages: [{ name: expect.any(String), version: expect.any(String) }],

packages/angular/src/sdk.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@ export function getDefaultIntegrations(_options: BrowserOptions = {}): Integrati
3333
// - https://github.com/getsentry/sentry-javascript/issues/5417#issuecomment-1453407097
3434
// - https://github.com/getsentry/sentry-javascript/issues/2744
3535
return [
36+
// TODO(v10): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration`
37+
// eslint-disable-next-line deprecation/deprecation
3638
inboundFiltersIntegration(),
3739
functionToStringIntegration(),
3840
breadcrumbsIntegration(),

packages/astro/src/index.server.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,9 @@ export {
5858
graphqlIntegration,
5959
hapiIntegration,
6060
httpIntegration,
61+
// eslint-disable-next-line deprecation/deprecation
6162
inboundFiltersIntegration,
63+
eventFiltersIntegration,
6264
initOpenTelemetry,
6365
isInitialized,
6466
kafkaIntegration,

packages/aws-serverless/src/index.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,9 @@ export {
5454
localVariablesIntegration,
5555
requestDataIntegration,
5656
functionToStringIntegration,
57+
// eslint-disable-next-line deprecation/deprecation
5758
inboundFiltersIntegration,
59+
eventFiltersIntegration,
5860
linkedErrorsIntegration,
5961
setMeasurement,
6062
getActiveSpan,

packages/browser/src/exports.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,9 @@ export {
4949
withScope,
5050
withIsolationScope,
5151
functionToStringIntegration,
52+
// eslint-disable-next-line deprecation/deprecation
5253
inboundFiltersIntegration,
54+
eventFiltersIntegration,
5355
dedupeIntegration,
5456
parameterize,
5557
startSession,

packages/browser/src/sdk.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,8 @@ export function getDefaultIntegrations(_options: Options): Integration[] {
3434
* `getDefaultIntegrations` but with an adjusted set of integrations.
3535
*/
3636
return [
37+
// TODO(v10): Replace with `eventFiltersIntegration` once we remove the deprecated `inboundFiltersIntegration`
38+
// eslint-disable-next-line deprecation/deprecation
3739
inboundFiltersIntegration(),
3840
functionToStringIntegration(),
3941
browserApiErrorsIntegration(),

packages/browser/test/index.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
77

88
import {
99
SDK_VERSION,
10+
eventFiltersIntegration,
1011
getGlobalScope,
1112
getIsolationScope,
1213
getReportDialogEndpoint,
13-
inboundFiltersIntegration,
1414
lastEventId,
1515
} from '@sentry/core';
1616
import * as utils from '@sentry/core';
@@ -304,12 +304,12 @@ describe('SentryBrowser', () => {
304304
expect(localBeforeSend).toHaveBeenCalledTimes(2);
305305
});
306306

307-
it('should use inboundfilter rules of bound client', async () => {
307+
it('should use eventFilters rules of bound client', async () => {
308308
const localBeforeSend = vi.fn();
309309
const options = getDefaultBrowserClientOptions({
310310
beforeSend: localBeforeSend,
311311
dsn,
312-
integrations: [inboundFiltersIntegration({ ignoreErrors: ['capture'] })],
312+
integrations: [eventFiltersIntegration({ ignoreErrors: ['capture'] })],
313313
});
314314
const client = new BrowserClient(options);
315315
setCurrentClient(client);

packages/bun/src/index.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,9 @@ export {
7474
requestDataIntegration,
7575
fsIntegration,
7676
functionToStringIntegration,
77+
// eslint-disable-next-line deprecation/deprecation
7778
inboundFiltersIntegration,
79+
eventFiltersIntegration,
7880
linkedErrorsIntegration,
7981
setMeasurement,
8082
getActiveSpan,

packages/bun/src/sdk.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ export function getDefaultIntegrations(_options: Options): Integration[] {
2929
// We return a copy of the defaultIntegrations here to avoid mutating this
3030
return [
3131
// Common
32+
// TODO(v10): Replace with eventFiltersIntegration once we remove the deprecated `inboundFiltersIntegration`
33+
// eslint-disable-next-line deprecation/deprecation
3234
inboundFiltersIntegration(),
3335
functionToStringIntegration(),
3436
linkedErrorsIntegration(),

0 commit comments

Comments
 (0)