List Info

Thread: ActionController request domain/subdomains bug (Ticket 9479)




ActionController request domain/subdomains bug (Ticket 9479)
country flaguser name
United States
2007-11-06 21:42:55
A while back I tripped over a case where ActionController
flakes when
handed a dotted-quad hostname behaves oddly; it (properly)
asserts
that the domain name is nil, but happily tells you the
subdomains are
the first two quads. Like so:

> console
>> request = ActionController::TestRequest.new
=> #<ActionController::TestRequest:0x38a2868 ...>
>> request.host = '192.168.1.1'
=> "192.168.1.1"
>> request.domain
=> nil
>> request.subdomains
=> ["192", "168"]

When patched, request.subdomains returns an empty
array.

Ticket and patch (with tests): http://dev.rub
yonrails.org/ticket/9479

Could I beg a quick review and, if worthy, a few +1s to get
this into
Rails 2?

Thanks!

--j


--~--~---------~--~----~------------~-------~--~----~
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-coregooglegroups.com
To unsubscribe from this group, send email to
rubyonrails-core-unsubscribegooglegroups.com
For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---


Re: ActionController request domain/subdomains bug (Ticket 9479)
country flaguser name
United States
2007-11-07 08:58:06
> Could I beg a quick review and, if worthy, a few +1s to
get this into
> Rails 2?

Good stuff. Applied.


--~--~---------~--~----~------------~-------~--~----~
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-coregooglegroups.com
To unsubscribe from this group, send email to
rubyonrails-core-unsubscribegooglegroups.com
For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---


[1-2]

about | contact  Other archives ( Real Estate discussion Medical topics )