feat: Show dropped event url in black/whitelist debug mode

Author: kamilogorek

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## Unreleased
+
+- [browser] feat: Show dropped event url in blacklistUrl/whitelistUrl debug mode
+- [browser] feat: Use better event description instead of event_id for user-facing logs
+
 ## 4.0.0
 
 This is the release of our new SDKs, `@sentry/browser`, `@sentry/node`. While there are too many changes to list for

packages/browser/src/integrations/breadcrumbs.ts

@@ -1,8 +1,7 @@
 import { API, getCurrentHub, logger } from '@sentry/core';
 import { Integration, Severity } from '@sentry/types';
 import { isFunction, isString } from '@sentry/utils/is';
-import { getGlobalObject, parseUrl } from '@sentry/utils/misc';
-import { getEventDescription } from '@sentry/utils/misc';
+import { getEventDescription, getGlobalObject, parseUrl } from '@sentry/utils/misc';
 import { deserialize, fill } from '@sentry/utils/object';
 import { safeJoin } from '@sentry/utils/string';
 import { supportsBeacon, supportsHistory, supportsNativeFetch } from '@sentry/utils/supports';

packages/browser/src/integrations/inboundfilters.ts

@@ -1,8 +1,8 @@
 import { configureScope, logger } from '@sentry/core';
 import { Integration, SentryEvent } from '@sentry/types';
 import { isRegExp } from '@sentry/utils/is';
-import { BrowserOptions } from '../backend';
 import { getEventDescription } from '@sentry/utils/misc';
+import { BrowserOptions } from '../backend';
 
 // "Script error." is hard coded into browsers for errors that it can't read.
 // this is the result of a script being pulled in from an external domain and CORS.
@@ -40,15 +40,25 @@ export class InboundFilters implements Integration {
   /** JSDoc */
   public shouldDropEvent(event: SentryEvent): boolean {
     if (this.isIgnoredError(event)) {
-      logger.warn(`Event dropped due to being matched by \`ignoreErrors\` option.\n  Event: ${getEventDescription(event)}`);
+      logger.warn(
+        `Event dropped due to being matched by \`ignoreErrors\` option.\nEvent: ${getEventDescription(event)}`,
+      );
       return true;
     }
     if (this.isBlacklistedUrl(event)) {
-      logger.warn(`Event dropped due to being matched by \`blacklistUrls\` option.\n  Event: ${getEventDescription(event)}`);
+      logger.warn(
+        `Event dropped due to being matched by \`blacklistUrls\` option.\nEvent: ${getEventDescription(
+          event,
+        )}.\nUrl: ${this.getEventFilterUrl(event)}`,
+      );
       return true;
     }
     if (!this.isWhitelistedUrl(event)) {
-      logger.warn(`Event dropped due to not being matched by \`whitelistUrls\` option.\n  Event: ${getEventDescription(event)}`);
+      logger.warn(
+        `Event dropped due to not being matched by \`whitelistUrls\` option.\nEvent: ${getEventDescription(
+          event,
+        )}.\nUrl: ${this.getEventFilterUrl(event)}`,
+      );
       return true;
     }
     return false;