Skip to content

chore: temporarily disable Sentry denyUrls to debug "Load failed" errors #15958

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 6, 2025

Conversation

iskounen
Copy link
Contributor

@iskounen iskounen commented Aug 6, 2025

Problem

We're seeing frequent "TypeError: Load failed" errors in Sentry but lack visibility into which third-party services might be causing them. Our current Sentry configuration blocks errors related to external services (Braze, Google Tag Manager, DoubleClick, etc.) via the denyUrls filter.

Solution

Temporarily disable the denyUrls configuration in Sentry client setup to allow third-party service errors (occurring on our domain) to reach Sentry.

I will monitor Sentry for 24 hours to collect data and then revert this change.

What this reveals

  • Script loading failures when artsy.net tries to load external services
  • SDK initialization errors from third-party services on our pages
  • Network errors from external service calls
  • Which specific services correlate with "Load failed" error patterns

Impact

  • No functional changes - same third-party services will still load/fail as before
  • Increased Sentry visibility into previously filtered errors
  • Temporary increase in error volume (expected)
  • Better debugging data to identify root cause of "Load failed" errors

cc: @artsy/diamond-devs

- Comment out denyUrls configuration in Sentry client setup
- This allows third-party service errors (on artsy.net domain) to reach Sentry
- Previously blocked: braze.com, doubleclick.net, googletagservices.com related errors
- Will help identify which external services cause "TypeError: Load failed"
- Temporary change for debugging - will re-enable after analysis

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@iskounen iskounen requested review from dzucconi and rabidpaul August 6, 2025 11:19
@iskounen iskounen self-assigned this Aug 6, 2025
@iskounen iskounen merged commit e58186c into main Aug 6, 2025
11 checks passed
@iskounen iskounen deleted the iskounen/chore/type-error-disable-sentry-deny-list branch August 6, 2025 11:46
@artsy-peril artsy-peril bot mentioned this pull request Aug 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant