{"issue":{"state":"closed","pull_request":{"diff_url":"https://github.com/rails/rails/pull/3631.diff","html_url":"https://github.com/rails/rails/pull/3631","patch_url":"https://github.com/rails/rails/pull/3631.patch"},"milestone":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/fb389f1e8b98d5d03be29e9dd309b3be?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"fb389f1e8b98d5d03be29e9dd309b3be","url":"https://api.github.com/users/indirect","login":"indirect","id":78},"labels":[],"assignee":null,"updated_at":"2011-11-14T22:30:56Z","comments":2,"closed_at":"2011-11-14T21:27:36Z","url":"https://api.github.com/repos/rails/rails/issues/3631","body":"This is a fix for the bug that @jonleighton found in 317f4e2, where remote_ip would sometimes return nil when it ought to have returned \"127.0.0.1\" (or any other IP that matches the \"trusted proxies to ignore\" regex).","html_url":"https://github.com/rails/rails/issues/3631","title":"Force calculation of remote IP before using || operator","number":3631,"id":2236188,"created_at":"2011-11-14T21:26:20Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/fb389f1e8b98d5d03be29e9dd309b3be?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"fb389f1e8b98d5d03be29e9dd309b3be","url":"https://api.github.com/users/indirect","login":"indirect","id":78},"updated_at":"2011-11-14T22:30:56Z","url":"https://api.github.com/repos/rails/rails/issues/comments/2737769","body":"Done in indirect/rails@b8c85de.","id":2737769,"created_at":"2011-11-14T22:30:56Z"}}