Skip to content
This repository has been archived by the owner on Jan 22, 2020. It is now read-only.

Updated dependencies. #56

Merged
merged 1 commit into from
Nov 23, 2017
Merged

Updated dependencies. #56

merged 1 commit into from
Nov 23, 2017

Conversation

joerg84
Copy link

@joerg84 joerg84 commented Nov 22, 2017

No description provided.

Gemfile.lock Outdated
execjs (>= 0.3.0, < 3)
unicode-display_width (1.0.3)
unicode-display_width (1.3.0)

PLATFORMS
ruby

DEPENDENCIES
cassandra-driver (~> 3.0)
rails (= 4.1.14.2)

Choose a reason for hiding this comment

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

This seems to not satisfy the reports for CVE-2016-6316 and CVE-2015-7581.
No wonder however, as we pin the rails version in Gemfile: https://github.com/mesosphere/tweeter/blob/master/Gemfile#L4 which should be changed and committed as well

@MrMarvin
Copy link

Generally we should try to use version pinning in Gemfile as much as possible and let bundler generate the lock-file from that. Can you please change and commit the Gemfile as well, @joerg84 ?

@joerg84
Copy link
Author

joerg84 commented Nov 23, 2017

@MrMarvin PTAL

Copy link

@MrMarvin MrMarvin left a comment

Choose a reason for hiding this comment

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

Looks good to me! Also quickly running brakeman on this branch does not show up additional serious vulnerabilities (two minor CSRF things, which I assume are not too serious for our use case).

@joerg84 joerg84 merged commit cccfb6e into master Nov 23, 2017
@joerg84 joerg84 deleted the dep branch November 23, 2017 11:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants