2008-01-05 08:32:06 -05:00
|
|
|
require 'abstract_unit'
|
2007-02-21 04:17:38 -05:00
|
|
|
require 'stringio'
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
class CookieStoreTest < ActionController::IntegrationTest
|
|
|
|
SessionKey = '_myapp_session'
|
|
|
|
SessionSecret = 'b3c631c314c0bbca50c1b2843150fe33'
|
2007-11-21 16:31:45 -05:00
|
|
|
|
2008-12-18 12:33:53 -05:00
|
|
|
Verifier = ActiveSupport::MessageVerifier.new(SessionSecret, 'SHA1')
|
2009-04-26 15:33:57 -04:00
|
|
|
SignedBar = Verifier.generate(:foo => "bar", :session_id => ActiveSupport::SecureRandom.hex(16))
|
2007-11-21 16:31:45 -05:00
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
class TestController < ActionController::Base
|
|
|
|
def no_session_access
|
|
|
|
head :ok
|
|
|
|
end
|
2007-02-21 04:17:38 -05:00
|
|
|
|
2008-12-18 12:33:53 -05:00
|
|
|
def persistent_session_id
|
|
|
|
render :text => session[:session_id]
|
|
|
|
end
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
def set_session_value
|
|
|
|
session[:foo] = "bar"
|
2009-02-13 21:08:43 -05:00
|
|
|
render :text => Rack::Utils.escape(Verifier.generate(session.to_hash))
|
2007-02-21 04:17:38 -05:00
|
|
|
end
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
def get_session_value
|
|
|
|
render :text => "foo: #{session[:foo].inspect}"
|
|
|
|
end
|
2007-02-21 04:17:38 -05:00
|
|
|
|
2009-02-07 00:15:39 -05:00
|
|
|
def get_session_id
|
2009-09-03 12:50:01 -04:00
|
|
|
render :text => "id: #{request.session_options[:id]}"
|
2009-02-07 00:15:39 -05:00
|
|
|
end
|
|
|
|
|
2008-12-20 15:37:51 -05:00
|
|
|
def call_reset_session
|
|
|
|
reset_session
|
|
|
|
head :ok
|
|
|
|
end
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
def raise_data_overflow
|
|
|
|
session[:foo] = 'bye!' * 1024
|
|
|
|
head :ok
|
|
|
|
end
|
2010-06-11 03:21:12 -04:00
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
def rescue_action(e) raise end
|
2007-02-21 04:17:38 -05:00
|
|
|
end
|
|
|
|
|
2007-03-13 16:44:16 -04:00
|
|
|
def test_raises_argument_error_if_missing_session_key
|
2008-12-15 17:33:31 -05:00
|
|
|
assert_raise(ArgumentError, nil.inspect) {
|
2009-01-27 19:54:01 -05:00
|
|
|
ActionDispatch::Session::CookieStore.new(nil,
|
2008-12-15 17:33:31 -05:00
|
|
|
:key => nil, :secret => SessionSecret)
|
|
|
|
}
|
|
|
|
|
|
|
|
assert_raise(ArgumentError, ''.inspect) {
|
2009-01-27 19:54:01 -05:00
|
|
|
ActionDispatch::Session::CookieStore.new(nil,
|
2008-12-15 17:33:31 -05:00
|
|
|
:key => '', :secret => SessionSecret)
|
|
|
|
}
|
2007-03-13 16:44:16 -04:00
|
|
|
end
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
def test_setting_session_value
|
|
|
|
with_test_route_set do
|
|
|
|
get '/set_session_value'
|
|
|
|
assert_response :success
|
2009-02-07 17:18:09 -05:00
|
|
|
assert_equal "_myapp_session=#{response.body}; path=/; HttpOnly",
|
2008-12-15 17:33:31 -05:00
|
|
|
headers['Set-Cookie']
|
2010-05-17 19:43:06 -04:00
|
|
|
end
|
2007-03-03 03:18:30 -05:00
|
|
|
end
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
def test_getting_session_value
|
|
|
|
with_test_route_set do
|
|
|
|
cookies[SessionKey] = SignedBar
|
|
|
|
get '/get_session_value'
|
|
|
|
assert_response :success
|
|
|
|
assert_equal 'foo: "bar"', response.body
|
2010-05-17 19:43:06 -04:00
|
|
|
end
|
2007-02-25 11:35:24 -05:00
|
|
|
end
|
|
|
|
|
2009-02-07 00:15:39 -05:00
|
|
|
def test_getting_session_id
|
|
|
|
with_test_route_set do
|
|
|
|
cookies[SessionKey] = SignedBar
|
|
|
|
get '/persistent_session_id'
|
|
|
|
assert_response :success
|
|
|
|
assert_equal response.body.size, 32
|
|
|
|
session_id = response.body
|
|
|
|
|
|
|
|
get '/get_session_id'
|
|
|
|
assert_response :success
|
2010-06-22 09:55:50 -04:00
|
|
|
assert_equal "id: #{session_id}", response.body, "should be able to read session id without accessing the session hash"
|
2009-02-07 00:15:39 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
def test_disregards_tampered_sessions
|
|
|
|
with_test_route_set do
|
|
|
|
cookies[SessionKey] = "BAh7BjoIZm9vIghiYXI%3D--123456780"
|
|
|
|
get '/get_session_value'
|
|
|
|
assert_response :success
|
|
|
|
assert_equal 'foo: nil', response.body
|
2007-03-03 03:18:30 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
def test_close_raises_when_data_overflows
|
2008-12-15 17:33:31 -05:00
|
|
|
with_test_route_set do
|
2010-05-17 19:43:06 -04:00
|
|
|
assert_raise(ActionDispatch::Cookies::CookieOverflow) {
|
2008-12-15 17:33:31 -05:00
|
|
|
get '/raise_data_overflow'
|
|
|
|
}
|
2008-09-16 12:22:11 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
def test_doesnt_write_session_cookie_if_session_is_not_accessed
|
|
|
|
with_test_route_set do
|
|
|
|
get '/no_session_access'
|
|
|
|
assert_response :success
|
2010-01-16 18:21:46 -05:00
|
|
|
assert_equal nil, headers['Set-Cookie']
|
2008-09-16 12:22:11 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
def test_doesnt_write_session_cookie_if_session_is_unchanged
|
|
|
|
with_test_route_set do
|
|
|
|
cookies[SessionKey] = "BAh7BjoIZm9vIghiYXI%3D--" +
|
|
|
|
"fef868465920f415f2c0652d6910d3af288a0367"
|
|
|
|
get '/no_session_access'
|
|
|
|
assert_response :success
|
2010-01-16 18:21:46 -05:00
|
|
|
assert_equal nil, headers['Set-Cookie']
|
2007-03-14 07:33:10 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2008-12-20 15:37:51 -05:00
|
|
|
def test_setting_session_value_after_session_reset
|
|
|
|
with_test_route_set do
|
|
|
|
get '/set_session_value'
|
|
|
|
assert_response :success
|
2008-12-25 07:10:28 -05:00
|
|
|
session_payload = response.body
|
2009-02-07 17:18:09 -05:00
|
|
|
assert_equal "_myapp_session=#{response.body}; path=/; HttpOnly",
|
2008-12-20 15:37:51 -05:00
|
|
|
headers['Set-Cookie']
|
|
|
|
|
|
|
|
get '/call_reset_session'
|
|
|
|
assert_response :success
|
|
|
|
assert_not_equal [], headers['Set-Cookie']
|
|
|
|
assert_not_equal session_payload, cookies[SessionKey]
|
|
|
|
|
|
|
|
get '/get_session_value'
|
|
|
|
assert_response :success
|
|
|
|
assert_equal 'foo: nil', response.body
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2010-06-22 09:55:50 -04:00
|
|
|
def test_getting_from_nonexistent_session
|
|
|
|
with_test_route_set do
|
|
|
|
get '/get_session_value'
|
|
|
|
assert_response :success
|
|
|
|
assert_equal 'foo: nil', response.body
|
|
|
|
assert_nil headers['Set-Cookie'], "should only create session on write, not read"
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2008-12-18 12:33:53 -05:00
|
|
|
def test_persistent_session_id
|
|
|
|
with_test_route_set do
|
|
|
|
cookies[SessionKey] = SignedBar
|
|
|
|
get '/persistent_session_id'
|
|
|
|
assert_response :success
|
|
|
|
assert_equal response.body.size, 32
|
|
|
|
session_id = response.body
|
|
|
|
get '/persistent_session_id'
|
|
|
|
assert_equal session_id, response.body
|
|
|
|
reset!
|
|
|
|
get '/persistent_session_id'
|
|
|
|
assert_not_equal session_id, response.body
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2009-01-28 00:05:07 -05:00
|
|
|
def test_session_store_with_expire_after
|
2009-09-26 13:56:53 -04:00
|
|
|
with_test_route_set(:expire_after => 5.hours) do
|
2009-01-28 00:05:07 -05:00
|
|
|
# First request accesses the session
|
|
|
|
time = Time.local(2008, 4, 24)
|
|
|
|
Time.stubs(:now).returns(time)
|
|
|
|
expected_expiry = (time + 5.hours).gmtime.strftime("%a, %d-%b-%Y %H:%M:%S GMT")
|
|
|
|
|
|
|
|
cookies[SessionKey] = SignedBar
|
|
|
|
|
|
|
|
get '/set_session_value'
|
|
|
|
assert_response :success
|
|
|
|
|
|
|
|
cookie_body = response.body
|
2009-04-13 18:18:45 -04:00
|
|
|
assert_equal "_myapp_session=#{cookie_body}; path=/; expires=#{expected_expiry}; HttpOnly",
|
|
|
|
headers['Set-Cookie']
|
2009-01-28 00:05:07 -05:00
|
|
|
|
|
|
|
# Second request does not access the session
|
|
|
|
time = Time.local(2008, 4, 25)
|
|
|
|
Time.stubs(:now).returns(time)
|
|
|
|
expected_expiry = (time + 5.hours).gmtime.strftime("%a, %d-%b-%Y %H:%M:%S GMT")
|
|
|
|
|
|
|
|
get '/no_session_access'
|
|
|
|
assert_response :success
|
|
|
|
|
2010-05-17 19:43:06 -04:00
|
|
|
assert_equal "_myapp_session=#{cookie_body}; path=/; expires=#{expected_expiry}; HttpOnly",
|
|
|
|
headers['Set-Cookie']
|
2009-01-28 00:05:07 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2010-06-11 01:55:39 -04:00
|
|
|
def test_session_store_with_explicit_domain
|
|
|
|
with_test_route_set(:domain => "example.es") do
|
|
|
|
get '/set_session_value'
|
|
|
|
assert_match /domain=example\.es/, headers['Set-Cookie']
|
|
|
|
headers['Set-Cookie']
|
|
|
|
end
|
|
|
|
end
|
2010-06-11 03:21:12 -04:00
|
|
|
|
2010-06-11 01:55:39 -04:00
|
|
|
def test_session_store_without_domain
|
|
|
|
with_test_route_set do
|
|
|
|
get '/set_session_value'
|
2010-06-24 14:02:23 -04:00
|
|
|
assert_no_match(/domain\=/, headers['Set-Cookie'])
|
2010-06-11 01:55:39 -04:00
|
|
|
end
|
|
|
|
end
|
2010-06-11 03:21:12 -04:00
|
|
|
|
2010-06-11 01:55:39 -04:00
|
|
|
def test_session_store_with_nil_domain
|
|
|
|
with_test_route_set(:domain => nil) do
|
|
|
|
get '/set_session_value'
|
2010-06-24 14:02:23 -04:00
|
|
|
assert_no_match(/domain\=/, headers['Set-Cookie'])
|
2010-06-11 01:55:39 -04:00
|
|
|
end
|
|
|
|
end
|
2010-06-11 03:21:12 -04:00
|
|
|
|
2010-06-11 01:55:39 -04:00
|
|
|
def test_session_store_with_all_domains
|
|
|
|
with_test_route_set(:domain => :all) do
|
|
|
|
get '/set_session_value'
|
2010-06-24 14:02:23 -04:00
|
|
|
assert_match(/domain=\.example\.com/, headers['Set-Cookie'])
|
2010-06-11 01:55:39 -04:00
|
|
|
end
|
|
|
|
end
|
2010-06-11 03:21:12 -04:00
|
|
|
|
2007-02-21 04:17:38 -05:00
|
|
|
private
|
2010-05-17 19:43:06 -04:00
|
|
|
|
|
|
|
# Overwrite get to send SessionSecret in env hash
|
|
|
|
def get(path, parameters = nil, env = {})
|
|
|
|
env["action_dispatch.secret_token"] ||= SessionSecret
|
|
|
|
super
|
|
|
|
end
|
|
|
|
|
2009-09-26 13:56:53 -04:00
|
|
|
def with_test_route_set(options = {})
|
2008-12-15 17:33:31 -05:00
|
|
|
with_routing do |set|
|
|
|
|
set.draw do |map|
|
2009-11-04 18:25:15 -05:00
|
|
|
match ':action', :to => ::CookieStoreTest::TestController
|
2008-12-15 17:33:31 -05:00
|
|
|
end
|
2010-05-17 19:43:06 -04:00
|
|
|
|
2010-05-17 21:18:23 -04:00
|
|
|
options = { :key => SessionKey }.merge!(options)
|
2010-05-17 19:43:06 -04:00
|
|
|
|
|
|
|
@app = self.class.build_app(set) do |middleware|
|
|
|
|
middleware.use ActionDispatch::Session::CookieStore, options
|
|
|
|
middleware.delete "ActionDispatch::ShowExceptions"
|
|
|
|
end
|
|
|
|
|
2008-12-15 17:33:31 -05:00
|
|
|
yield
|
2007-02-21 04:17:38 -05:00
|
|
|
end
|
|
|
|
end
|
2007-03-03 08:54:54 -05:00
|
|
|
end
|