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 May 11, 2021
1 parent 867f18f commit e0ffa1b
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 @@ -192,6 +183,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.20.1)
parser (3.0.1.1)
ast (~> 2.4.1)
Expand Down Expand Up @@ -378,7 +372,7 @@ DEPENDENCIES
memcachier
oj
omniauth-facebook
omniauth-rails!
omniauth-rails_csrf_protection
pg
pry-rails
puma
Expand Down

0 comments on commit e0ffa1b

Please sign in to comment.