rails/rails — pull request #3640 by indirect (2011-11-15 21:34:50+00:00)
Title: GetIp#to_s should never return nil. That's icky.

Okay, so my previous fix for this was to acknowledge that GetIp#to_s could return nil. And that's really, really awful. So this fix returns the first IP value in REMOTE_ADDR instead of nil, and stops assuming that to_s could return nil. All the tests pass, and I'd welcome any feedback on how to make this better/cleaner. Thanks!