Permalink
Browse files

Deprecate legacy CGI options in SessionStores.

  • Loading branch information...
1 parent bb2327d commit 94878c61a329891eb904ace5b06dbc50831869d8 @josevalim josevalim committed May 18, 2010
View
6 actionpack/lib/action_controller/session/abstract_store.rb
@@ -98,12 +98,18 @@ def initialize(app, options = {})
# Process legacy CGI options
options = options.symbolize_keys
if options.has_key?(:session_path)
+ ActiveSupport::Deprecation.warn "Giving :session_path to SessionStore is deprecated, " <<
+ "please use :path instead", caller
options[:path] = options.delete(:session_path)
end
if options.has_key?(:session_key)
+ ActiveSupport::Deprecation.warn "Giving :session_key to SessionStore is deprecated, " <<
+ "please use :key instead", caller
options[:key] = options.delete(:session_key)
end
if options.has_key?(:session_http_only)
+ ActiveSupport::Deprecation.warn "Giving :session_http_only to SessionStore is deprecated, " <<
+ "please use :httponly instead", caller
options[:httponly] = options.delete(:session_http_only)
end
View
6 actionpack/lib/action_controller/session/cookie_store.rb
@@ -59,12 +59,18 @@ def initialize(app, options = {})
# Process legacy CGI options
options = options.symbolize_keys
if options.has_key?(:session_path)
+ ActiveSupport::Deprecation.warn "Giving :session_path to SessionStore is deprecated, " <<
+ "please use :path instead", caller
options[:path] = options.delete(:session_path)
end
if options.has_key?(:session_key)
+ ActiveSupport::Deprecation.warn "Giving :session_key to SessionStore is deprecated, " <<
+ "please use :key instead", caller
options[:key] = options.delete(:session_key)
end
if options.has_key?(:session_http_only)
+ ActiveSupport::Deprecation.warn "Giving :session_http_only to SessionStore is deprecated, " <<
+ "please use :httponly instead", caller
options[:httponly] = options.delete(:session_http_only)
end

0 comments on commit 94878c6

Please sign in to comment.