heroku-bouncer 0.4.2 → 0.4.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  SHA1:
3
- metadata.gz: 828f65b68c8ed42f82cf523ab3748a16263fe5d2
4
- data.tar.gz: 6b53e6ef3efb5f53bd35af410e0fd0501a3a9722
3
+ metadata.gz: 415f3f61030a94fb1e1eb0a4481be6fb8d60b5cd
4
+ data.tar.gz: 7cc105448d3d48f17ac0271ce336bb355245631e
5
5
  SHA512:
6
- metadata.gz: 0b317ac658086d8ebe4157ab5b85b93fd23b61569d0ac5b41c7426a9514c4b490d06ffabc2daed835a40fcd2a33bd33812afe95d4d0843b6438d6d17b0894c4a
7
- data.tar.gz: 12fd8a29894b24568fbfd0ed6d236b640cb4658084692e61f5900e7dc32ca69ebab3570d3760c76cef8ddf3f88ca6b45d27d6cbb6ef0fb5a6af75451b41d9c44
6
+ metadata.gz: c977a0ea26bdac8fd14863a761a34492f9155b341171f707544bb9a816032f82bc29ba5414f0013168bcab71801a8ef0dcca56f20caea249a9b89eac9f577063
7
+ data.tar.gz: aa0ce2396d42163f876f37c9f3ae01c7d831250f26e664411a8aa56782101d065582c3351b705256c5bd322d6c99b2cd5a14e0b5a3cb755a5155b98c227d7e20
@@ -7,7 +7,7 @@ class Heroku::Bouncer::Builder
7
7
  def self.new(app, options = {})
8
8
  builder = Rack::Builder.new
9
9
  id, secret, scope = extract_options!(options)
10
- unless id.empty? || secret.empty?
10
+ unless options[:disabled]
11
11
  builder.use OmniAuth::Builder do
12
12
  provider :heroku, id, secret, :scope => scope
13
13
  end
@@ -17,10 +17,10 @@ class Heroku::Bouncer::Builder
17
17
  end
18
18
 
19
19
  def self.extract_options!(options)
20
- oauth = options.delete(:oauth) || {}
21
- id = oauth.delete(:id)
22
- secret = oauth.delete(:secret)
23
- scope = oauth.delete(:scope) || 'identity'
20
+ oauth = options[:oauth] || {}
21
+ id = oauth[:id]
22
+ secret = oauth[:secret]
23
+ scope = oauth[:scope] || 'identity'
24
24
 
25
25
  if id.nil? && (ENV.has_key?('HEROKU_ID') || ENV.has_key?('HEROKU_OAUTH_ID'))
26
26
  $stderr.puts "[warn] heroku-bouncer: HEROKU_ID or HEROKU_OAUTH_ID detected in environment, please pass in :oauth hash instead"
@@ -32,8 +32,8 @@ class Heroku::Bouncer::Builder
32
32
  secret = ENV['HEROKU_OAUTH_SECRET'] || ENV['HEROKU_SECRET']
33
33
  end
34
34
 
35
- if id.nil? || secret.nil?
36
- $stderr.puts "[fatal] heroku-bouncer: HEROKU_OAUTH_ID or HEROKU_OAUTH_SECRET not set, middleware disabled"
35
+ if id.nil? || secret.nil? || id.empty? || secret.empty?
36
+ $stderr.puts "[fatal] heroku-bouncer: OAuth ID or secret not set, middleware disabled"
37
37
  options[:disabled] = true
38
38
  end
39
39
 
@@ -172,8 +172,10 @@ private
172
172
  end
173
173
 
174
174
  def encrypt_store(env)
175
- env["rack.session"][:bouncer] =
176
- env["rack.session"][:bouncer].lock(@cookie_secret)
175
+ if env["rack.session"].key?(:bouncer)
176
+ env["rack.session"][:bouncer] =
177
+ env["rack.session"][:bouncer].lock(@cookie_secret)
178
+ end
177
179
  end
178
180
 
179
181
  def store
metadata CHANGED
@@ -1,14 +1,14 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: heroku-bouncer
3
3
  version: !ruby/object:Gem::Version
4
- version: 0.4.2
4
+ version: 0.4.3
5
5
  platform: ruby
6
6
  authors:
7
7
  - Jonathan Dance
8
8
  autorequire:
9
9
  bindir: bin
10
10
  cert_chain: []
11
- date: 2014-05-07 00:00:00.000000000 Z
11
+ date: 2014-05-27 00:00:00.000000000 Z
12
12
  dependencies:
13
13
  - !ruby/object:Gem::Dependency
14
14
  name: omniauth-heroku