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

Regex error in amplify.store.js #65

Closed
ycrumeyrolle opened this issue Jul 11, 2012 · 0 comments
Closed

Regex error in amplify.store.js #65

ycrumeyrolle opened this issue Jul 11, 2012 · 0 comments

Comments

@ycrumeyrolle
Copy link

At line 201 in amplify.store.js, the regex used to sanitize the key is incorrect.

Unicode character \u37f is incorrect an should be corrected as \u037f

key = key.replace( /[^-._0-9A-Za-z\xb7\xc0-\xd6\xd8-\xf6\xf8-\u037d\u37f-\u1fff\u200c-\u200d\u203f\u2040\u2070-\u218f]/g, "-" );

should be
key = key.replace( /[^-._0-9A-Za-z\xb7\xc0-\xd6\xd8-\xf6\xf8-\u037d\u037f-\u1fff\u200c-\u200d\u203f\u2040\u2070-\u218f]/g, "-" );

This regex cause some parsing errors, such in IDE Visual Studio or with bundling feature in ASP.NET MVC4.

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