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

Check RubyGems Vulnerabilities #179

Closed
TamerShlash opened this issue Sep 4, 2017 · 12 comments
Closed

Check RubyGems Vulnerabilities #179

TamerShlash opened this issue Sep 4, 2017 · 12 comments

Comments

@TamerShlash
Copy link

Greetings,

Mainly inspired by "Multiple vulnerabilities in RubyGems" blog post on official Ruby website, I was thinking whether bundler-audit should check for vulnerabilities in RubyGems itself or not.

It would be great if it did in my opinion.

@modosc
Copy link

modosc commented Sep 15, 2017

i think it would if the vulns were merged into the db but that pr is still open:

rubysec/ruby-advisory-db#298

@errm
Copy link

errm commented Sep 20, 2017

There is also this project https://github.com/civisanalytics/ruby_audit that can check rubygems and the current ruby against the ruby-advisory-db

@modosc
Copy link

modosc commented Sep 20, 2017

thanks @errm - it's dead though, no updates in a while.

1 similar comment
@modosc
Copy link

modosc commented Sep 20, 2017

thanks @errm - it's dead though, no updates in a while.

@errm
Copy link

errm commented Sep 21, 2017

It does work though... and since is using the ruby-advisory-db is as up to date as that is... there don't seem to be any open issues so I guess its just inactive because it just works :)
Perhaps @jeffreyc could comment on it's current state?

@jeffreyc
Copy link

@errm is correct: it continues to work and no one's filed any issues, so there's been no need for any changes (though I should update it to use bundler-audit 0.6.0). It actually started as #118, which needs some love, but could be updated to add this behavior to bundler-audit, if the rubysec team wanted to go in that direction.

@postmodern
Copy link
Member

postmodern commented Oct 12, 2017

It would require adding special logic for handling rubygems vulnerabilities, since rubygems comes vendored (but not as a gem) with ruby and can also be updated via installing the rubygems-update gem. I'm assuming this feature would just have to check vulnerable version ranges against Gem::VERSION and instruct the user to run gem update --system? I think it's fine for ruby_audit to take on this feature, as bundler-audit purpose is to audit your bundled projects/apps, not necessarily your runtime/development environment.

@ericsullivan
Copy link

You can specify a ruby version in bundler, which creates an entry in Gemfile.lock. Should bundler-audit evaluate that entry for vulnerabilities?
ruby_audit doesn't use that version (it calls ruby --version).

@postmodern
Copy link
Member

@ericsullivan that would be more useful than just checking the current ruby version bundler-audit happens to be running under. However, bundler doesn't record or require specific rubygems versions. So even if you are running a patched version of rubygems, your production environment may not be; and vice versa.

@ericsullivan
Copy link

Right, it's only half of a solution. If you specify a ruby version in the Gemfile and it doesn't match bundler throws an error and doesn't start, so it is a nice way to ensure your app is running on the correct ruby version. That you can't specify rubygems is unfortunate.

@ericsullivan
Copy link

As an update, specifying the RubyGems version in bundler is not a feature the team would like to take on at this time.

@TamerShlash
Copy link
Author

I guess I understand now why this does not belong here.

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

6 participants