-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
JQuery 1.12.4 (with vuln) loaded by default #3905
Comments
Thanks for pointing this out! |
@jarednorman has there been any movement or attempt at resolution to this? |
Not that I know of. We should definitely upgrade and I suspect it wouldn't be too difficult, but I think only the first two could realistically affect the app, and we audited the code in the early days after the fork when we found an XSS vulnerability that was similar but involved select2. |
I would like to report that I had to update to It will be nice if the jQuery version gets updated to a recent, supported version so new users of solidus will not have to face the same issue. |
@cpfergus1 seems to have fixed this already. @kennyadsl I would assume it is safe to close this issue. |
The version of jQuery loaded by default on all (frontend, backend) Solidus apps is version 1.12.4, as that is the version that
jquery-rails
loads when requiring simplyjquery
. This version has several known security vulnerabilities. The use of jQuery 1.12.4 was recently flagged for us as a security vulnerability, and was a surprise, as we don't use that version of jQuery.A simple fix might be to update to
jquery3
in all places wherejquery
is required, but we are unsure if this will have broader consequences. Also, we were unsure if this was on your radar, as it seems to be a simple thing to miss.Solidus Version:
Latest
To Reproduce
Generate a new Solidus app, see the jquery version pulled in
The text was updated successfully, but these errors were encountered: