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

Jquery.js #1591

Open
jano-01 opened this issue Oct 16, 2024 · 3 comments
Open

Jquery.js #1591

jano-01 opened this issue Oct 16, 2024 · 3 comments

Comments

@jano-01
Copy link

jano-01 commented Oct 16, 2024

Hello!

This more of a security issue and a question to that. Why are you using version 1.7.1 of jquery? Everything less than 1.9 has an XSS vulnerability?

Best wishes!
Jan

@adam12
Copy link
Contributor

adam12 commented Oct 16, 2024

Discussed elsewhere on the repo. Here's one place that I recall: #1351 (comment)

@noraj
Copy link

noraj commented Oct 20, 2024

Why are you using version 1.7.1 of jquery?

Seems it was upgraded to 3.4.1 (9ed7586) to fix a privious vulnerability and then reverted 4fbf8ff

Everything less than 1.9 has an XSS vulnerability?

Every repository hosting yard generated doc is receiving GHSA-q4m3-2j7h-f7xw alerts. Ex: https://github.com/noraj/PixelChart/security/dependabot/15

I guess yard is probably not vulnerable as it's to generate static documentation and there is nearly no user input outside of the search bar. But that's annoying that anyone using yard receives a false positive vulnerability alert.

@lsegal
Copy link
Owner

lsegal commented Oct 20, 2024

The upgraded version of jQuery created regressions in generated documentation. If someone wants to open a PR with a version of jQuery that does not break downstream usage, it might be accepted. Another option would be for someone to provide the upgrade with necessary shims/updates to yard code in order to not break downstream users. Alternatively, if someone wants to show a proper reproduction of the vulnerability being used in the context of yard, it could be prioritized more highly.

It's worth noting that downstream users are also free to provide their own jQuery by using templating in yard to override the original version, so if you're getting notices on GitHub, you can address them by vendoring your own jQuery; you will however be subject to potential regressions.

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

4 participants