Firefox source url fix #20
Conversation
|
I put testMode in because there are issues with Raven.getHeaders on local files. I believe it's needed for some browsers in order for the tests to run. I'll double check that. The rest of this change looks great, though. Thank you! |
Fetching HTTP headers on the file:https:/// scheme doesn't work, thus the testMode. Here we're just being more precise about it.
And potentially other browsers.
|
I see. It must be IE, I couldn't reproduce the error in Chrom/FF/Safari/Opera under OSX. I pushed another change instead that just disables the getHeader function instead of the whole HTTP interface. |
|
Okay, after some testing in Chrome, Safari, Firefox, Opera, and IE9, here are the quirks: Firefox, Opera, and IE9 do not support origin, so it should be dropped. IE9 does not support href. All 5 support protocol + host + pathname, so that's what I'll move Raven to. I'll merge this in and tweak the url. Thanks for the contribution! |
Make the source url compatible with more browsers, and allow the Http interface in test mode (without getting headers)
Report correct origin url in FF