From 0f048a5c9867b2e991ad540222c8d8188f5a24fc Mon Sep 17 00:00:00 2001 From: Jeremy Kemper Date: Mon, 1 Jan 2007 01:13:09 +0000 Subject: Set session to an empty hash if :new_session => false and no session cookie or param is present. CGI::Session was raising an unrescued ArgumentError. git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@5820 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- actionpack/CHANGELOG | 2 ++ actionpack/lib/action_controller/cgi_process.rb | 8 +++++++- 2 files changed, 9 insertions(+), 1 deletion(-) (limited to 'actionpack') diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG index f303154f31..edc28f2027 100644 --- a/actionpack/CHANGELOG +++ b/actionpack/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Set session to an empty hash if :new_session => false and no session cookie or param is present. CGI::Session was raising an unrescued ArgumentError. [Josh Susser] + * Routing uses URI escaping for path components and CGI escaping for query parameters. [darix, Jeremy Kemper] * Fix assert_redirected_to bug where redirecting from a nested to to a top-level controller incorrectly added the current controller's nesting. Closes #6128. [Rick Olson] diff --git a/actionpack/lib/action_controller/cgi_process.rb b/actionpack/lib/action_controller/cgi_process.rb index 47de1956cc..1ee0d60e07 100644 --- a/actionpack/lib/action_controller/cgi_process.rb +++ b/actionpack/lib/action_controller/cgi_process.rb @@ -110,7 +110,13 @@ module ActionController #:nodoc: if session_options_with_string_keys['new_session'] == true @session = new_session else - @session = CGI::Session.new(@cgi, session_options_with_string_keys) + begin + @session = CGI::Session.new(@cgi, session_options_with_string_keys) + # CGI::Session raises ArgumentError if 'new_session' == false + # and no session cookie or query param is present. + rescue ArgumentError + @session = Hash.new + end end @session['__valid_session'] end -- cgit v1.2.3