Permalink
Browse files

Merge pull request #6084 from brainopia/support_for_magic_domain_on_a…

…ll_stores

Support cookie jar options for all cookie stores
  • Loading branch information...
2 parents 09de707 + 4121938 commit bca9a9e926db11c5c9a438493d49317146c64fb6 @josevalim josevalim committed Apr 30, 2012
@@ -74,6 +74,13 @@ def stale_session_check!
class AbstractStore < Rack::Session::Abstract::ID
include Compatibility
include StaleSessionCheck
+
+ private
+
+ def set_cookie(env, session_id, cookie)
+ request = ActionDispatch::Request.new(env)
+ request.cookie_jar[key] = cookie
+ end
end
end
end
@@ -254,6 +254,13 @@ def test_incoming_invalid_session_id_via_parameter_should_be_ignored
end
end
+ def test_session_store_with_all_domains
+ with_test_route_set(:domain => :all) do
+ get '/set_session_value'
+ assert_response :success
+ end
+ end
+
private
def with_test_route_set(options = {})

0 comments on commit bca9a9e

Please sign in to comment.