Skip to content

Commit

Permalink
Update Nokogiri for a security patch
Browse files Browse the repository at this point in the history
Problem:

Running `bundler-audit` reveals a security vulnerability in Nokogiri,
which can be traced back to a libxml2 vulnerability.

```
$ bundle-audit
Name: nokogiri
Version: 1.6.6.2
Advisory: CVE-2015-1819
Criticality: Unknown
URL: sparklemotion/nokogiri#1374
Title: Nokogiri gem contains several vulnerabilities in libxml2 and libxslt
Solution: upgrade to ~> 1.6.6.4, >= 1.6.7.rc4

Vulnerabilities found!
```

Solution:

Update nokogiri to `~> 1.6.6.4`, as suggested.

Nokogiri is a dependency of capybara,
which is a dependency of poltergeist.

We only need to specify the nokogiri version for the test bundler group,
which already depends on it.

References:

https://github.com/rubysec/bundler-audit
sparklemotion/nokogiri#1374
http://www.ubuntu.com/usn/usn-2812-1/

Only use nokogiri in test bundler group

Nokogiri is a dependency of capybara,
which is a dependency of poltergeist.
We only need to specify nokogiri for bundler groups that already depend
on it.
  • Loading branch information
c-lliope committed Dec 5, 2015
1 parent 3e0acb2 commit 12e0db4
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion Gemfile
Expand Up @@ -28,12 +28,13 @@ end

group :test do
gem "ammeter"
gem "poltergeist"
gem "database_cleaner"
gem "formulaic"
gem "fuubar"
gem "launchy"
gem "nokogiri", "~> 1.6.6.4"
gem "percy-capybara"
gem "poltergeist"
gem "shoulda-matchers", "~> 2.8.0", require: false
gem "timecop"
gem "webmock"
Expand Down
3 changes: 2 additions & 1 deletion Gemfile.lock
Expand Up @@ -167,7 +167,7 @@ GEM
neat (1.7.2)
bourbon (>= 4.0)
sass (>= 3.3)
nokogiri (1.6.6.2)
nokogiri (1.6.6.4)
mini_portile (~> 0.6.0)
normalize-rails (3.0.3)
percy-capybara (1.0.0)
Expand Down Expand Up @@ -310,6 +310,7 @@ DEPENDENCIES
high_voltage
i18n-tasks
launchy
nokogiri (~> 1.6.6.4)
percy-capybara
pg
poltergeist
Expand Down

0 comments on commit 12e0db4

Please sign in to comment.