Skip to content

Commit

Permalink
Use omniauth-rails_csrf_protection gem
Browse files Browse the repository at this point in the history
It seems like omniauth-rails is not actually the correct gem:

  omniauth/omniauth-rails#2 (comment)

Here's the relevant instructions:

  https://github.com/omniauth/omniauth/wiki/Resolving-CVE-2015-9284
  • Loading branch information
dgmstuart committed Nov 17, 2019
1 parent 64456c1 commit a87b859
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 11 deletions.
2 changes: 1 addition & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ gem 'jbuilder'
gem 'jquery-rails'
gem 'memcachier'
gem 'omniauth-facebook'
gem 'omniauth-rails', git: 'https://github.com/omniauth/omniauth-rails', branch: 'CVE-2015-9284'
gem 'omniauth-rails_csrf_protection'
gem 'pg'
gem 'pry-rails'
gem 'puma'
Expand Down
14 changes: 4 additions & 10 deletions Gemfile.lock
Original file line number Diff line number Diff line change
@@ -1,12 +1,3 @@
GIT
remote: https://github.com/omniauth/omniauth-rails
revision: 8ef80e7da0b4b12dd403ba579b0a34dd6efebdae
branch: CVE-2015-9284
specs:
omniauth-rails (1.0.0)
omniauth (~> 1.0)
rails

GEM
remote: https://rubygems.org/
specs:
Expand Down Expand Up @@ -191,6 +182,9 @@ GEM
omniauth-oauth2 (1.5.0)
oauth2 (~> 1.1)
omniauth (~> 1.2)
omniauth-rails_csrf_protection (0.1.2)
actionpack (>= 4.2)
omniauth (>= 1.3.1)
parallel (1.19.0)
parser (2.6.5.0)
ast (~> 2.4.0)
Expand Down Expand Up @@ -368,7 +362,7 @@ DEPENDENCIES
memcachier
oj
omniauth-facebook
omniauth-rails!
omniauth-rails_csrf_protection
pg
pry-rails
puma
Expand Down

0 comments on commit a87b859

Please sign in to comment.