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

Security Warning - Cross-Site Scripting #211

Closed
pandu-cls opened this issue Jul 14, 2021 · 3 comments
Closed

Security Warning - Cross-Site Scripting #211

pandu-cls opened this issue Jul 14, 2021 · 3 comments

Comments

@pandu-cls
Copy link

We are using Loofah as a dependency on some other gem. When it got upgraded to latest version 2.10.0 our brakeman is showing a security warning (Cross-Site Scripting). Please take a look at below for more details.

image

We are using below versions.
Rails - 6.1.3.2
Ruby - 2.7.4
Brakeman - 4.8.2

Please let us know if any resolution is made or we have to downgrade to 2.9.1

@flavorjones
Copy link
Owner

@pandu-cls Thanks for asking this question. I believe this is a bug in brakeman that has been fixed if you upgrade to the latest version. See these issues for deeper context:

@flavorjones
Copy link
Owner

I've pinned #209 to hopefully help others who see this.

@pandu-cls
Copy link
Author

@flavorjones Thank you very much for quick response. We have upgraded our brakeman version and it start passing now. Appreciated 👍

va-bot pushed a commit to department-of-veterans-affairs/caseflow that referenced this issue Jul 20, 2021
Snooze the jquery-rails gem security warning while we work on updating it safely
See this PR for updating the gem: #16514
Updating jquery-rails updates loofah to a version that revealed a bug in brakeman. See: flavorjones/loofah#211
Updating brakeman is going to be a more complicated step so snoozing to unblock prs for now
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