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

[CIRCLE-37398] Fix for CVE-2020-8165 (activesupport) #5636

Merged
merged 1 commit into from
Aug 31, 2021

Conversation

snyk-bot
Copy link
Contributor

Snyk has created this PR to fix one or more vulnerable packages in the `rubygems` dependencies of this project.

Changes included in this PR

  • Changes to the following files to upgrade the vulnerable dependencies to a fixed version:
    • src-api/Gemfile
⚠️ Warning
Failed to update the Gemfile.lock, please update manually before merging.

Vulnerabilities that will be fixed

With an upgrade:
Severity Priority Score (*) Issue Breaking Change Exploit Maturity
high severity 834/1000
Why? Mature exploit, Has a fix available, CVSS 8.1
Deserialization of Untrusted Data
SNYK-RUBY-ACTIVESUPPORT-569598
No Mature

(*) Note that the real score may have changed since the PR was raised.

Check the changes in this PR to ensure they won't cause issues with your project.


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

🛠 Adjust project settings

📚 Read more about Snyk's upgrade and patch logic

@rsylvian rsylvian marked this pull request as draft August 24, 2021 02:51
@rsylvian rsylvian changed the title [Snyk] Fix for 1 vulnerabilities [CIRCLE-37398] Fix for CVE-2020-8165 (activesupport) Aug 24, 2021
@nramabad nramabad force-pushed the snyk-fix-3cc5be386c8af04cea84f9618b761c27 branch from c71ff04 to 2723aa5 Compare August 25, 2021 17:53
@nramabad nramabad marked this pull request as ready for review August 25, 2021 17:53
src-api/config.rb Outdated Show resolved Hide resolved
gem 'middleman-sprockets', '~> 4.1.0'
gem 'rouge', '~> 2.0.5'
gem 'redcarpet', '~> 3.4.0'
gem 'nokogiri', '~> 1.10.8'
gem "therubyracer"
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 unnecessary nowadays and adds a large memory footprint not to mention source of build failure

https://kmitov.com/posts/we-dont-need-therubyracer-and-the-libv8-error-for-compiling-native-extensions/

…ry therubyracer & comment out sprockets

The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/SNYK-RUBY-ACTIVESUPPORT-569598
@nramabad nramabad force-pushed the snyk-fix-3cc5be386c8af04cea84f9618b761c27 branch from 2723aa5 to ebe1a51 Compare August 25, 2021 19:57
Copy link
Contributor

@rsylvian rsylvian left a comment

Choose a reason for hiding this comment

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

LGTM but how do we make sure it still works?

@nramabad
Copy link
Contributor

LGTM but how do we make sure it still works?

@teesloane you got any ideas?

@teesloane
Copy link
Contributor

We should clone this locally test it, and then rename the branch to a *-preview branch, push it, and make sure it works on our preview deployment. I will do this today.

@teesloane
Copy link
Contributor

The output artifact of v1 (src-api) can be viewed here.

@nramabad nramabad merged commit ca6f307 into master Aug 31, 2021
@nramabad nramabad deleted the snyk-fix-3cc5be386c8af04cea84f9618b761c27 branch August 31, 2021 19:55
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.

None yet

4 participants