getsentry · AJDoubleW · Jun 3, 2020 · Jun 4, 2020 · Jun 4, 2020 · Jun 4, 2020
diff --git a/MIGRATION.md b/MIGRATION.md
@@ -255,6 +255,8 @@ Sentry.addBreadcrumb({
 
 > 'ignoreUrls' was renamed to 'blacklistUrls'. 'ignoreErrors', which has a similar name was not renamed. [Docs](https://docs.sentry.io/error-reporting/configuration/?platform=browser#blacklist-urls) and [Decluttering Sentry](https://docs.sentry.io/platforms/javascript/#decluttering-sentry)
 
+> 'blacklistUrls' has since been renamed to 'excludedUrls'
+
 _Old_:
 
 ```js
@@ -270,7 +272,7 @@ _New_:
 
 ```js
 Sentry.init({
-  blacklistUrls: [
+  excludedUrls: [
     'https://www.baddomain.com',
     /graph\.facebook\.com/i,
   ],

diff --git a/packages/browser/examples/app.js b/packages/browser/examples/app.js
@@ -37,9 +37,9 @@ Sentry.init({
   // An array of strings or regexps that'll be used to ignore specific errors based on their type/message
   ignoreErrors: [/PickleRick_\d\d/, 'RangeError'],
   // An array of strings or regexps that'll be used to ignore specific errors based on their origin url
-  blacklistUrls: ['external-lib.js'],
+  excludedUrls: ['external-lib.js'],
   // An array of strings or regexps that'll be used to allow specific errors based on their origin url
-  whitelistUrls: ['http://localhost:5000', 'https://browser.sentry-cdn'],
+  includedUrls: ['http://localhost:5000', 'https://browser.sentry-cdn'],
   // Debug mode with valuable initialization/lifecycle informations.
   debug: true,
   // Whether SDK should be enabled or not.
@@ -93,15 +93,15 @@ Sentry.init({
 // Testing code, irrelevant vvvvv
 
 document.addEventListener('DOMContentLoaded', () => {
-  document.querySelector('#blacklist-url').addEventListener('click', () => {
+  document.querySelector('#excluded-url').addEventListener('click', () => {
     const script = document.createElement('script');
     script.crossOrigin = 'anonymous';
     script.src =
       'https://rawgit.com/kamilogorek/cfbe9f92196c6c61053b28b2d42e2f5d/raw/3aef6ff5e2fd2ad4a84205cd71e2496a445ebe1d/external-lib.js';
     document.body.appendChild(script);
   });
 
-  document.querySelector('#whitelist-url').addEventListener('click', () => {
+  document.querySelector('#included-url').addEventListener('click', () => {
     const script = document.createElement('script');
     script.crossOrigin = 'anonymous';
     script.src =

diff --git a/packages/browser/examples/index.html b/packages/browser/examples/index.html
@@ -20,8 +20,8 @@
     </style>
   </head>
   <body>
-    <button id="blacklist-url">blacklistUrl example</button>
-    <button id="whitelist-url">whitelistUrl example</button>
+    <button id="excluded-url">excludedUrl example</button>
+    <button id="included-url">includedUrl example</button>
     <button id="ignore-message">ignoreError message example</button>
     <button id="ignore-type">ignoreError type example</button>
     <button id="regular-exception">regularException example</button>

diff --git a/packages/browser/src/backend.ts b/packages/browser/src/backend.ts
@@ -12,17 +12,17 @@ import { FetchTransport, XHRTransport } from './transports';
 export interface BrowserOptions extends Options {
   /**
    * A pattern for error URLs which should not be sent to Sentry.
-   * To whitelist certain errors instead, use {@link Options.whitelistUrls}.
+   * To include certain errors instead, use {@link Options.includedUrls}.
    * By default, all errors will be sent.
    */
-  blacklistUrls?: Array<string | RegExp>;
+  excludedUrls?: Array<string | RegExp>;
 
   /**
    * A pattern for error URLs which should exclusively be sent to Sentry.
-   * This is the opposite of {@link Options.blacklistUrls}.
+   * This is the opposite of {@link Options.excludedUrls}.
    * By default, all errors will be sent.
    */
-  whitelistUrls?: Array<string | RegExp>;
+  includedUrls?: Array<string | RegExp>;
 }
 
 /**

diff --git a/packages/browser/test/integration/common/init.js b/packages/browser/test/integration/common/init.js
@@ -34,13 +34,13 @@ function initSDK() {
     integrations: [new Sentry.Integrations.Dedupe()],
     attachStacktrace: true,
     ignoreErrors: ["ignoreErrorTest"],
-    blacklistUrls: ["foo.js"],
-    beforeSend: function(event, eventHint) {
+    excludedUrls: ["foo.js"],
+    beforeSend: function (event, eventHint) {
       events.push(event);
       eventHints.push(eventHint);
       return event;
     },
-    beforeBreadcrumb: function(breadcrumb, breadcrumbHint) {
+    beforeBreadcrumb: function (breadcrumb, breadcrumbHint) {
       // Filter console logs as we use them for debugging *a lot* and they are not *that* important
       // But allow then if we explicitly say so (for one of integration tests)
       if (

diff --git a/packages/browser/test/integration/suites/config.js b/packages/browser/test/integration/suites/config.js
@@ -21,10 +21,10 @@ describe("config", function() {
        * > bar.js file called a function in baz.js
        * > baz.js threw an error
        *
-       * foo.js is blacklisted in the `init` call (init.js), thus we filter it
+       * foo.js is excluded in the `init` call (init.js), thus we filter it
        * */
-      var urlWithBlacklistedUrl = new Error("filter");
-      urlWithBlacklistedUrl.stack =
+      var urlWithExcludedUrl = new Error("filter");
+      urlWithExcludedUrl.stack =
         "Error: bar\n" +
         " at http://localhost:5000/foo.js:7:19\n" +
         " at bar(http://localhost:5000/bar.js:2:3)\n" +
@@ -35,17 +35,17 @@ describe("config", function() {
        * > bar-pass.js file called a function in baz-pass.js
        * > baz-pass.js threw an error
        *
-       * foo-pass.js is *not* blacklisted in the `init` call (init.js), thus we don't filter it
+       * foo-pass.js is *not* excluded in the `init` call (init.js), thus we don't filter it
        * */
-      var urlWithoutBlacklistedUrl = new Error("pass");
-      urlWithoutBlacklistedUrl.stack =
+      var urlWithoutExcludedUrl = new Error("pass");
+      urlWithoutExcludedUrl.stack =
         "Error: bar\n" +
         " at http://localhost:5000/foo-pass.js:7:19\n" +
         " at bar(http://localhost:5000/bar-pass.js:2:3)\n" +
         " at baz(http://localhost:5000/baz-pass.js:2:9)\n";
 
-      Sentry.captureException(urlWithBlacklistedUrl);
-      Sentry.captureException(urlWithoutBlacklistedUrl);
+      Sentry.captureException(urlWithExcludedUrl);
+      Sentry.captureException(urlWithoutExcludedUrl);
     }).then(function(summary) {
       assert.lengthOf(summary.events, 1);
       assert.equal(summary.events[0].exception.values[0].type, "Error");

diff --git a/packages/core/src/integrations/inboundfilters.ts b/packages/core/src/integrations/inboundfilters.ts
@@ -8,10 +8,10 @@ const DEFAULT_IGNORE_ERRORS = [/^Script error\.?$/, /^Javascript error: Script e
 
 /** JSDoc */
 interface InboundFiltersOptions {
-  blacklistUrls?: Array<string | RegExp>;
+  excludedUrls?: Array<string | RegExp>;
   ignoreErrors?: Array<string | RegExp>;
   ignoreInternal?: boolean;
-  whitelistUrls?: Array<string | RegExp>;
+  includedUrls?: Array<string | RegExp>;
 }
 
 /** Inbound filters configurable by the user */
@@ -61,17 +61,17 @@ export class InboundFilters implements Integration {
       );
       return true;
     }
-    if (this._isBlacklistedUrl(event, options)) {
+    if (this._isExcludedUrl(event, options)) {
       logger.warn(
-        `Event dropped due to being matched by \`blacklistUrls\` option.\nEvent: ${getEventDescription(
+        `Event dropped due to being matched by \`excludedUrls\` option.\nEvent: ${getEventDescription(
           event,
         )}.\nUrl: ${this._getEventFilterUrl(event)}`,
       );
       return true;
     }
-    if (!this._isWhitelistedUrl(event, options)) {
+    if (!this._isIncludedUrl(event, options)) {
       logger.warn(
-        `Event dropped due to not being matched by \`whitelistUrls\` option.\nEvent: ${getEventDescription(
+        `Event dropped due to not being matched by \`includedUrls\` option.\nEvent: ${getEventDescription(
           event,
         )}.\nUrl: ${this._getEventFilterUrl(event)}`,
       );
@@ -113,36 +113,36 @@ export class InboundFilters implements Integration {
   }
 
   /** JSDoc */
-  private _isBlacklistedUrl(event: Event, options: InboundFiltersOptions = {}): boolean {
+  private _isExcludedUrl(event: Event, options: InboundFiltersOptions = {}): boolean {
     // TODO: Use Glob instead?
-    if (!options.blacklistUrls || !options.blacklistUrls.length) {
+    if (!options.excludedUrls || !options.excludedUrls.length) {
       return false;
     }
     const url = this._getEventFilterUrl(event);
-    return !url ? false : options.blacklistUrls.some(pattern => isMatchingPattern(url, pattern));
+    return !url ? false : options.excludedUrls.some(pattern => isMatchingPattern(url, pattern));
   }
 
   /** JSDoc */
-  private _isWhitelistedUrl(event: Event, options: InboundFiltersOptions = {}): boolean {
+  private _isIncludedUrl(event: Event, options: InboundFiltersOptions = {}): boolean {
     // TODO: Use Glob instead?
-    if (!options.whitelistUrls || !options.whitelistUrls.length) {
+    if (!options.includedUrls || !options.includedUrls.length) {
       return true;
     }
     const url = this._getEventFilterUrl(event);
-    return !url ? true : options.whitelistUrls.some(pattern => isMatchingPattern(url, pattern));
+    return !url ? true : options.includedUrls.some(pattern => isMatchingPattern(url, pattern));
   }
 
   /** JSDoc */
   private _mergeOptions(clientOptions: InboundFiltersOptions = {}): InboundFiltersOptions {
     return {
-      blacklistUrls: [...(this._options.blacklistUrls || []), ...(clientOptions.blacklistUrls || [])],
+      excludedUrls: [...(this._options.excludedUrls || []), ...(clientOptions.excludedUrls || [])],
       ignoreErrors: [
         ...(this._options.ignoreErrors || []),
         ...(clientOptions.ignoreErrors || []),
         ...DEFAULT_IGNORE_ERRORS,
       ],
       ignoreInternal: typeof this._options.ignoreInternal !== 'undefined' ? this._options.ignoreInternal : true,
-      whitelistUrls: [...(this._options.whitelistUrls || []), ...(clientOptions.whitelistUrls || [])],
+      includedUrls: [...(this._options.includedUrls || []), ...(clientOptions.includedUrls || [])],
     };
   }