Skip to content
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

"Javascript error: Script error. on line 0" #223

Closed
skovhus opened this issue Jul 2, 2014 · 9 comments
Closed

"Javascript error: Script error. on line 0" #223

skovhus opened this issue Jul 2, 2014 · 9 comments

Comments

@skovhus
Copy link

skovhus commented Jul 2, 2014

First, thanks for a great library!

I'm wondering about an event we are getting with the message Javascript error: Script error. on line 0. Our whitelistUrls is only including our own js files.

Currently I see these events coming from different browsers:

30 Firefox 30.0
30 Chrome 35.0.1916.153
8 Safari 6.1.4
6 Safari 6.0
3 Safari 7.0.4
2 ChromeiOS 35.0.1916.41
2 Chrome 35.0.1916.114
2 Safari 7.0.3
1 Chrome 34.0.1847.131
1 Chrome 33.0.1750.154

In the source code Raven.js already set up filters for Script error
https://github.com/getsentry/raven-js/blob/master/src/raven.js#L66

But should Javascript error: Script error. on line 0 also be added by default?

I guess the event I'm seeing is related "script being pulled in from an external domain and CORS" as described in http://blog.errorception.com/2012/04/script-error-on-line-0.html

@mattrobenolt
Copy link
Contributor

@skovhus What is the exact error message being sent? Is it a different case or something?

@skovhus
Copy link
Author

skovhus commented Jul 2, 2014

@mattrobenolt so I cannot reproduce the error, but as you see some users with different browsers get this error. The message shown in Sentry is Javascript error: Script error. on line 0, so I guess that's the exact error message. And as the Script error in L66 is not a regexp, our error is not the ignoreErrors list.

@skovhus
Copy link
Author

skovhus commented Jul 2, 2014

@mattrobenolt But I guess I can get the message from our nginx if you want it. But shouldn't it be the same as reported by Sentry?

@mattrobenolt
Copy link
Contributor

Yeah, that's good enough. Are all those other browser being reported as this exact literal string? Or do you see other variants? If so, I'll add this one to the list.

@skovhus
Copy link
Author

skovhus commented Jul 2, 2014

I see these variations:
Javascript error: Script error on line 0 (IE)
Javascript error: Script error. on line 0 (all the other browsers)

@mattrobenolt
Copy link
Contributor

lol

@skovhus
Copy link
Author

skovhus commented Jul 2, 2014

For once, the feedback from IE seems sane... : )

@skovhus
Copy link
Author

skovhus commented Jul 7, 2014

#224

@mattrobenolt
Copy link
Contributor

Closed with #224

kamilogorek pushed a commit that referenced this issue Jun 12, 2018
* Disable client instantiation in errorHandler

* Use captureException the new way

* Move middleware to live on Raven prototype
billyvg added a commit that referenced this issue Dec 5, 2024
Includes the following fixes:

- fix: Catch calls to iframe content document ([#222](getsentry/rrweb#222))
- fix(snapshot): Fix CSS expansion of add CSS property ([#223](getsentry/rrweb#223))
billyvg added a commit that referenced this issue Dec 6, 2024
Includes the following fixes:

- fix: Catch calls to iframe content document
([#222](getsentry/rrweb#222))
- fix(snapshot): Fix CSS expansion of add CSS property
([#223](getsentry/rrweb#223))
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

No branches or pull requests

2 participants