Browse files

Revert "Revert "Merge pull request #6084 from brainopia/support_for_m…

…agic_domain_on_all_stores""

This reverts commit a48ea68.
  • Loading branch information...
1 parent ae2383d commit 058dc9a09680e9e2a5777c441b966a6f5e12cf82 @brainopia brainopia committed Aug 10, 2012
View
7 actionpack/lib/action_dispatch/middleware/session/abstract_store.rb
@@ -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
View
7 actionpack/test/activerecord/active_record_store_test.rb
@@ -256,6 +256,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 058dc9a

Please sign in to comment.