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

Downgrade Bundler on Travis #960

Merged
merged 1 commit into from Jan 11, 2019
Merged

Downgrade Bundler on Travis #960

merged 1 commit into from Jan 11, 2019

Conversation

mike-burns
Copy link
Contributor

@mike-burns mike-burns commented Jan 11, 2019

Bundler 2.0.1 works with Rails, but [the released version of
bundler-audit hardcodes a dependency on 1.x]. [Downgrade Bundler on Travis]
until [the new bundler-audit] is out.

[the released version of bundler-audit hardcodes a dependency on 1.x]: rubysec/bundler-audit#202
[Downgrade Bundler on Travis]: https://docs.travis-ci.com/user/languages/ruby/#bundler-20
[the new bundler-audit]: rubysec/bundler-audit#203
@mike-burns mike-burns merged commit a0b8511 into master Jan 11, 2019
@mike-burns mike-burns deleted the fix-for-bundler-2 branch January 11, 2019 16:44
@croaky
Copy link
Contributor

croaky commented Jan 11, 2019

Is Bundler Audit needed for projects using GitHub given their security alert feature? https://blog.github.com/2017-11-16-introducing-security-alerts-on-github/

Wondering if another approach would be to keep latest Bundler but remove Bundler Audit.

@mike-burns
Copy link
Contributor Author

@croaky ah interesting, hadn't considered that.

I'm going to bring that discussion to the rest of thoughtbot (#ruby and #security channels) and ask them to weigh in here, too.

@edwardloveall
Copy link
Contributor

By itself, I do like the idea of tests failing when dependencies have a security vulnerability. It's a nice, reactive way (after you run CI or local tests) for your environment to encourage, or even enforce, that your gems are updated. It combines nicely with the GitHub feature of proactively (before you even run the tests) warning you that you're using dependencies with known security holes.

Blocking bundler 2 and hardcoding for bundler 1.x is certainly a bummer though.

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