Skip to content

Deploy #15952

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 9 commits into from
Aug 6, 2025
Merged

Deploy #15952

merged 9 commits into from
Aug 6, 2025

Conversation

artsyit
Copy link
Contributor

@artsyit artsyit commented Aug 5, 2025

This is an automatically generated release PR!

erikdstock and others added 6 commits August 5, 2025 11:18
…rrors (#15957)

- Add script.onerror handler to capture external script loading failures
- Send structured error data to Sentry with scriptId and src URL
- Tag errors as "script_load_failure" for easy filtering
- Helps debug "TypeError: Load failed" errors by identifying which external scripts are failing

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

Co-authored-by: Claude <[email protected]>
…#15958)

- 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 merged commit 81ced27 into release Aug 6, 2025
9 checks passed
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.

5 participants