{"comment":{"url":"https://api.github.com/repos/Microsoft/Phoenix/comments/16513214","html_url":"https://github.com/Microsoft/Phoenix/commit/ce7e601ad1fcddcf6107755478ff4b5cfcc2af0f#commitcomment-16513214","id":16513214,"user":{"login":"raldaba","id":15915500,"avatar_url":"https://avatars.githubusercontent.com/u/15915500?v=3","gravatar_id":"","url":"https://api.github.com/users/raldaba","html_url":"https://github.com/raldaba","followers_url":"https://api.github.com/users/raldaba/followers","following_url":"https://api.github.com/users/raldaba/following{/other_user}","gists_url":"https://api.github.com/users/raldaba/gists{/gist_id}","starred_url":"https://api.github.com/users/raldaba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/raldaba/subscriptions","organizations_url":"https://api.github.com/users/raldaba/orgs","repos_url":"https://api.github.com/users/raldaba/repos","events_url":"https://api.github.com/users/raldaba/events{/privacy}","received_events_url":"https://api.github.com/users/raldaba/received_events","type":"User","site_admin":false},"position":29,"line":770,"path":"CMP/CmpWap/CmpWapExtension/TenantExtension/Controllers/CmpWapExtensionTenantController.cs","commit_id":"ce7e601ad1fcddcf6107755478ff4b5cfcc2af0f","created_at":"2016-03-04T23:23:05Z","updated_at":"2016-03-04T23:23:05Z","body":"I would suggest a conditional operation here via the ? operator to handle cases where the _recentIp is not determined. Might happen if the session expires, since this data is quite volatile.\r\n\r\nOn that note, minor suggestion: if you could rename it to _recentIp, that would be great, just to follow the Camel Case convention. No big deal, though, just pointing it out."}}