summary refs log tree commit
diff options
context:
space:
mode:
authorKonstantin Haase <konstantin.mailinglists@googlemail.com>2012-01-18 07:43:03 -0800
committerKonstantin Haase <konstantin.mailinglists@googlemail.com>2012-01-18 07:43:03 -0800
commit62ff9ad3d5733f913a4da895bd8f2493075c32e0 (patch)
tree843c383a61d6cb4893fcb59a720dd8f303733207
parent9a004149f61382d0bfc0a45b81a992db695e66d6 (diff)
parentd56434e01971bead101c6fd7fe5d1ecd6e2fe9fe (diff)
downloadrack-62ff9ad3d5733f913a4da895bd8f2493075c32e0.tar.gz
Merge pull request #317 from brainopia/master
A few stylistic changes
-rw-r--r--lib/rack/session/abstract/id.rb6
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/rack/session/abstract/id.rb b/lib/rack/session/abstract/id.rb
index 641e3c03..9e19d1da 100644
--- a/lib/rack/session/abstract/id.rb
+++ b/lib/rack/session/abstract/id.rb
@@ -36,7 +36,7 @@ module Rack
       private
 
         def session_id_not_loaded?
-          !key?(:id) && !@session_id_loaded
+          !(@session_id_loaded || key?(:id))
         end
 
         def load_session_id!
@@ -183,7 +183,7 @@ module Rack
           :renew =>         false,
           :sidbits =>       128,
           :cookie_only =>   true,
-          :secure_random => begin ::SecureRandom rescue false end
+          :secure_random => (::SecureRandom rescue false)
         }
 
         attr_reader :key, :default_options
@@ -191,7 +191,7 @@ module Rack
         def initialize(app, options={})
           @app = app
           @default_options = self.class::DEFAULT_OPTIONS.merge(options)
-          @key = options[:key] || "rack.session"
+          @key = @default_options.delete(:key)
           @cookie_only = @default_options.delete(:cookie_only)
           initialize_sid
         end