Rails core - Jul 2010 - Patch Review Request: #5147 Allowing any TLD format for the cookie_store domain in Rails 3

I''ve added a patch that fixes the new ":all" domain option
for cookie
session stores in Rails 3.  So, calls like this:

config.session_store :cookie_store, :key =>
''_mydevelopmentsite_session'', :domain => :all

will work on any style TLD.

I use a ".local" TLD on my development machine and the current
implementation does not allow for this. It will just fail to create
the session, causing ActionController::InvalidAuthenticityToken
exceptions.  Also, the current code does not work for "co.uk" or
"com.au".  My patch is much more flexible and should cover most TLDs.

https://rails.lighthouseapp.com/projects/8994/tickets/5147-the-all-domain-option-for-the-cookie-session-store-doesnt-allow-non-standard-tlds-like-local-or-couk

Would any of you be willing to verify the patch works for you?

Thanks,
Bryce

-- 
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Core" group.
To post to this group, send email to rubyonrails-core@googlegroups.com.
To unsubscribe from this group, send email to
rubyonrails-core+unsubscribe@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/rubyonrails-core?hl=en.