Skip to content

Remove jquery-migrate dependency#118

Open
zwik wants to merge 1 commit into
nytimes:masterfrom
zwik:master
Open

Remove jquery-migrate dependency#118
zwik wants to merge 1 commit into
nytimes:masterfrom
zwik:master

Conversation

@zwik

@zwik zwik commented Jun 15, 2015

Copy link
Copy Markdown

jquery-migrate-1.2.1.js is vulnerable to XSS attacks. This fixes removes the dependency of this file. You can now also upgrade your JQuery to a higher version.

jquery-migrate-1.2.1.js is vulnerable to XSS attacks. This fixes removes the dependency of this file. You can now also upgrade your JQuery to a higher version.
@zwik

zwik commented Jun 10, 2016

Copy link
Copy Markdown
Author

@itmilos yes, we run in production with this change :) .

@itmilos

itmilos commented Jun 27, 2016

Copy link
Copy Markdown

I have tested everything, it looks good I would definitely merge this into master.

@delambo

delambo commented Sep 23, 2016

Copy link
Copy Markdown
Contributor

Thanks @zwik @itmilos - Can you add a note under the Master changelog section about this change and the recommended version of jQuery (or at least the version you're testing against).

@delambo

delambo commented Sep 23, 2016

Copy link
Copy Markdown
Contributor

Fixes #87

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

Successfully merging this pull request may close these issues.

3 participants