Skip to content

allow this to get around the Firefox SecurityError issues #29

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

allow this to get around the Firefox SecurityError issues #29

wants to merge 1 commit into from

Conversation

xchoi
Copy link

@xchoi xchoi commented Nov 4, 2014

@marcj
Copy link
Owner

marcj commented Jan 13, 2015

Well, thanks, but that won't fix it in all cases since your string.indexOf check is not bulletproof.

@Starefossen
Copy link

We are seeing a lot of this error on our site from Firefox v34 users.
There are two solutions proposed in this post on SO.

@marcj
Copy link
Owner

marcj commented Jan 14, 2015

I guess the only fix for that is to use a try-catch.

@ElManouche
Copy link
Contributor

Why don't you add the try-catch ?

@marcj marcj closed this May 6, 2015
@marcj
Copy link
Owner

marcj commented May 6, 2015

Fixed in #52

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@xchoi @marcj @Starefossen @ElManouche