kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Merge pull request #17792 from rockrep/master

Browse source 
allow 'all' for :domain option in addition to :all
This commit is contained in:
Rafael Mendonça França 2014-12-04 15:21:47 -02:00
commit dfa45ce7e1
2 changed files with 11 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
actionpack/lib/action_dispatch/middleware/cookies.rb
View file

@ -283,7 +283,7 @@ module ActionDispatch
def handle_options(options) #:nodoc:
options[:path] ||= "/"
if options[:domain] == :all
if options[:domain].respond_to?(:to_sym) && options[:domain].to_sym == :all
# if there is a provided tld length then we use it otherwise default domain regexp
domain_regexp = options[:tld_length] ? /([^.]+\.?){#{options[:tld_length]}}$/ : DOMAIN_REGEXP

10
actionpack/test/dispatch/cookies_test.rb
View file

@ -145,11 +145,21 @@ class CookiesTest < ActionController::TestCase
head :ok
end
def set_cookie_with_domain_all_as_string
cookies[:user_name] = {:value => "rizwanreza", :domain => 'all'}
head :ok
end
def delete_cookie_with_domain
cookies.delete(:user_name, :domain => :all)
head :ok
end
def delete_cookie_with_domain_all_as_string
cookies.delete(:user_name, :domain => 'all')
head :ok
end
def set_cookie_with_domain_and_tld
cookies[:user_name] = {:value => "rizwanreza", :domain => :all, :tld_length => 2}
head :ok