Skip to content

[Snyk] Fix for 3 vulnerabilities#10768

Closed
pauldoomgov wants to merge 1 commit intomainfrom
snyk-fix-365f9c47d7b8864682d8ad23922620a4
Closed

[Snyk] Fix for 3 vulnerabilities#10768
pauldoomgov wants to merge 1 commit intomainfrom
snyk-fix-365f9c47d7b8864682d8ad23922620a4

Conversation

@pauldoomgov
Copy link
Contributor

This PR was automatically created by Snyk using the credentials of a real user.


![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)

Snyk has created this PR to fix 3 vulnerabilities in the rubygems dependencies of this project.

Snyk changed the following file(s):

  • Gemfile
⚠️ Warning ``` Failed to update the Gemfile.lock, please update manually before merging. ```

Vulnerabilities that will be fixed with an upgrade:

Issue Score
medium severity Cross-site Scripting (XSS)
SNYK-RUBY-ACTIONTEXT-7210236
  591  
medium severity Improper Input Validation
SNYK-RUBY-ACTIONPACK-7210237
  496  
medium severity Missing Cryptographic Step
SNYK-RUBY-OPENSSL-6036190
  479  

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Improper Input Validation
🦉 Cross-site Scripting (XSS)

gem 'saml_idp', github: '18F/saml_idp', tag: '0.21.0-18f'
gem 'scrypt'
gem 'simple_form', '>= 5.0.2'
gem 'simple_form', '>= 5.3.1'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a big jump, I would be curious to see if we could avoid upgrading this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah these upgrades look off. I would expect we'd only need to bump Rails, and there should also be corresponding Gemfile.lock changes included.

@zachmargolis
Copy link
Contributor

Closing in favor of #10770

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.

4 participants