Skip to content

Commit 50395bd

Browse files
AbhiPrasadlobsterkatie
authored andcommitted
feat(core): Remove whitelistUrls/blacklistUrls (#4850)
1 parent d2fe512 commit 50395bd

File tree

2 files changed

+2
-27
lines changed

2 files changed

+2
-27
lines changed

packages/browser/src/backend.ts

-6
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,6 @@ export interface BrowserOptions extends Options {
2323
* By default, all errors will be sent.
2424
*/
2525
denyUrls?: Array<string | RegExp>;
26-
27-
/** @deprecated use {@link Options.allowUrls} instead. */
28-
whitelistUrls?: Array<string | RegExp>;
29-
30-
/** @deprecated use {@link Options.denyUrls} instead. */
31-
blacklistUrls?: Array<string | RegExp>;
3226
}
3327

3428
/**

packages/core/src/integrations/inboundfilters.ts

+2-21
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,6 @@ export interface InboundFiltersOptions {
1313
denyUrls: Array<string | RegExp>;
1414
ignoreErrors: Array<string | RegExp>;
1515
ignoreInternal: boolean;
16-
17-
/** @deprecated use {@link InboundFiltersOptions.allowUrls} instead. */
18-
whitelistUrls: Array<string | RegExp>;
19-
/** @deprecated use {@link InboundFiltersOptions.denyUrls} instead. */
20-
blacklistUrls: Array<string | RegExp>;
2116
}
2217

2318
/** Inbound filters configurable by the user */
@@ -60,22 +55,8 @@ export function _mergeOptions(
6055
clientOptions: Partial<InboundFiltersOptions> = {},
6156
): Partial<InboundFiltersOptions> {
6257
return {
63-
allowUrls: [
64-
// eslint-disable-next-line deprecation/deprecation
65-
...(internalOptions.whitelistUrls || []),
66-
...(internalOptions.allowUrls || []),
67-
// eslint-disable-next-line deprecation/deprecation
68-
...(clientOptions.whitelistUrls || []),
69-
...(clientOptions.allowUrls || []),
70-
],
71-
denyUrls: [
72-
// eslint-disable-next-line deprecation/deprecation
73-
...(internalOptions.blacklistUrls || []),
74-
...(internalOptions.denyUrls || []),
75-
// eslint-disable-next-line deprecation/deprecation
76-
...(clientOptions.blacklistUrls || []),
77-
...(clientOptions.denyUrls || []),
78-
],
58+
allowUrls: [...(internalOptions.allowUrls || []), ...(clientOptions.allowUrls || [])],
59+
denyUrls: [...(internalOptions.denyUrls || []), ...(clientOptions.denyUrls || [])],
7960
ignoreErrors: [
8061
...(internalOptions.ignoreErrors || []),
8162
...(clientOptions.ignoreErrors || []),

0 commit comments

Comments
 (0)