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

Security Error on WebWorker IE11 Win8.1 #4358

Closed
michelson opened this issue Mar 2, 2017 · 6 comments
Closed

Security Error on WebWorker IE11 Win8.1 #4358

michelson opened this issue Mar 2, 2017 · 6 comments

Comments

@michelson
Copy link

Hello , we have found some security issues on windows explorer 11.
the specific error is on the blob url initialization for web worker. The bug is documented here. , it seems to be a bug on previous versions of IE11. The problem is that the mapboxgl.supported() still returns true. So, I see two options , mapboxgl.supported() should return false or patch WebWorker initialization with an strategy similar to https://github.com/parallel-js/parallel.js/pull/16/files

I think this is actually related with #2504

@michelson
Copy link
Author

michelson commented Mar 2, 2017

some references: we are using latest version of mapbox ( 0.32.1 )

image

image

@andrewharvey
Copy link
Collaborator

I don't think it's related to #2504, in that I was getting multiple "Unimplemented type: 3" errors because my version of IE11/specific set of windows patches had a broken gzip which meant that the pbf vector tiles weren't being uncompressed by the browser.

@michelson
Copy link
Author

hi @andrewharvey, I know, it's not strictly related, but is related in the sense that mapboxgl.support() is returning true, but the map throws an error.
we indeed thought that the error we had was the #2504, but then with some debugging we realise that it was a Web Worker blob issue.

@fabiomcosta
Copy link

fabiomcosta commented Apr 12, 2017

I just noticed the same issue.
Maps won't show up on IE11.
Debugging I could see that the error happens on the following line:

return new window.Worker(workerURL);

@andrewharvey does this helps? Let me know if you need more information.

screen shot 2017-04-12 at 3 34 00 pm

@anandthakker
Copy link
Contributor

Since this issue is fixed in current versions, I think fixing supported() to correctly return false should be sufficient.

@droachrkn
Copy link

droachrkn commented Mar 23, 2018

This issue has not been fixed. Using the latest map box GL, IE 11 still throws the same error. This is not related to anything within this chain or otherwise. "Security Error", debugger pauses on the WebWorker.

@anandthakker

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants