{"action":"created","comment":{"url":"https://api.github.com/repos/healthbam/healthbam/pulls/comments/85651692","pull_request_review_id":6357062,"id":85651692,"diff_hunk":"@@ -35,10 +50,20 @@ public String getCallerIp() {\n         return request.getRemoteAddr();\n     }\n \n+    @Timed\n     @Override\n-    public String getCurrentUser() {\n-        LOGGER.debug(\"getCurrentUser called\");\n-        return \"someone\";\n+    public String getCurrentUserEmail() {\n+        LOGGER.debug(\"getCurrentUserEmail called\");\n+\n+        HmhbUser loggedInUser = authorizationService.getLoggedInUser();\n+\n+        String email = null;\n+\n+        if (loggedInUser != null) {\n+            email = loggedInUser.getEmail();","path":"src/main/java/org/hmhb/audit/DefaultAuditHelper.java","position":58,"original_position":58,"commit_id":"4aff58455d3606d44f7e1cdff7612c59effdbb7e","original_commit_id":"4aff58455d3606d44f7e1cdff7612c59effdbb7e","user":{"login":"RyanBard","id":2766717,"avatar_url":"https://avatars.githubusercontent.com/u/2766717?v=3","gravatar_id":"","url":"https://api.github.com/users/RyanBard","html_url":"https://github.com/RyanBard","followers_url":"https://api.github.com/users/RyanBard/followers","following_url":"https://api.github.com/users/RyanBard/following{/other_user}","gists_url":"https://api.github.com/users/RyanBard/gists{/gist_id}","starred_url":"https://api.github.com/users/RyanBard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RyanBard/subscriptions","organizations_url":"https://api.github.com/users/RyanBard/orgs","repos_url":"https://api.github.com/users/RyanBard/repos","events_url":"https://api.github.com/users/RyanBard/events{/privacy}","received_events_url":"https://api.github.com/users/RyanBard/received_events","type":"User","site_admin":false},"body":"When adding authentication, I had forgotten to fill this in, so the audit data was still populating a hardcoded \"someone\".","created_at":"2016-10-30T02:03:27Z","updated_at":"2016-10-30T02:03:27Z","html_url":"https://github.com/healthbam/healthbam/pull/233#discussion_r85651692","pull_request_url":"https://api.github.com/repos/healthbam/healthbam/pulls/233","_links":{"self":{"href":"https://api.github.com/repos/healthbam/healthbam/pulls/comments/85651692"},"html":{"href":"https://github.com/healthbam/healthbam/pull/233#discussion_r85651692"},"pull_request":{"href":"https://api.github.com/repos/healthbam/healthbam/pulls/233"}}},"pull_request":{"url":"https://api.github.com/repos/healthbam/healthbam/pulls/233","id":91508170,"html_url":"https://github.com/healthbam/healthbam/pull/233","diff_url":"https://github.com/healthbam/healthbam/pull/233.diff","patch_url":"https://github.com/healthbam/healthbam/pull/233.patch","issue_url":"https://api.github.com/repos/healthbam/healthbam/issues/233","number":233,"state":"open","locked":false,"title":"closes #232: Update javadocs and cleanup","user":{"login":"RyanBard","id":2766717,"avatar_url":"https://avatars.githubusercontent.com/u/2766717?v=3","gravatar_id":"","url":"https://api.github.com/users/RyanBard","html_url":"https://github.com/RyanBard","followers_url":"https://api.github.com/users/RyanBard/followers","following_url":"https://api.github.com/users/RyanBard/following{/other_user}","gists_url":"https://api.github.com/users/RyanBard/gists{/gist_id}","starred_url":"https://api.github.com/users/RyanBard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RyanBard/subscriptions","organizations_url":"https://api.github.com/users/RyanBard/orgs","repos_url":"https://api.github.com/users/RyanBard/repos","events_url":"https://api.github.com/users/RyanBard/events{/privacy}","received_events_url":"https://api.github.com/users/RyanBard/received_events","type":"User","site_admin":false},"body":"* Adds missing @Timed annotations.\r\n* Adds missing javadocs.\r\n* Adds missing unit tests.\r\n* Adds missing logging.\r\n* Adds missing audit code.\r\n* Adds some missing validation code.\r\n* Removes some dead code.\r\n* Fixes ordering of county suggestions.\r\n* Cleans up JsonErrorResponse to be immutable.","created_at":"2016-10-30T01:52:12Z","updated_at":"2016-10-30T02:03:27Z","closed_at":null,"merged_at":null,"merge_commit_sha":"0c56d9516d92a1cb3b09999f5d082edf6299e6b2","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/healthbam/healthbam/pulls/233/commits","review_comments_url":"https://api.github.com/repos/healthbam/healthbam/pulls/233/comments","review_comment_url":"https://api.github.com/repos/healthbam/healthbam/pulls/comments{/number}","comments_url":"https://api.github.com/repos/healthbam/healthbam/issues/233/comments","statuses_url":"https://api.github.com/repos/healthbam/healthbam/statuses/4aff58455d3606d44f7e1cdff7612c59effdbb7e","head":{"label":"RyanBard:Issue232_UpdateDocs","ref":"Issue232_UpdateDocs","sha":"4aff58455d3606d44f7e1cdff7612c59effdbb7e","user":{"login":"RyanBard","id":2766717,"avatar_url":"https://avatars.githubusercontent.com/u/2766717?v=3","gravatar_id":"","url":"https://api.github.com/users/RyanBard","html_url":"https://github.com/RyanBard","followers_url":"https://api.github.com/users/RyanBard/followers","following_url":"https://api.github.com/users/RyanBard/following{/other_user}","gists_url":"https://api.github.com/users/RyanBard/gists{/gist_id}","starred_url":"https://api.github.com/users/RyanBard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RyanBard/subscriptions","organizations_url":"https://api.github.com/users/RyanBard/orgs","repos_url":"https://api.github.com/users/RyanBard/repos","events_url":"https://api.github.com/users/RyanBard/events{/privacy}","received_events_url":"https://api.github.com/users/RyanBard/received_events","type":"User","site_admin":false},"repo":{"id":67940746,"name":"healthbam","full_name":"RyanBard/healthbam","owner":{"login":"RyanBard","id":2766717,"avatar_url":"https://avatars.githubusercontent.com/u/2766717?v=3","gravatar_id":"","url":"https://api.github.com/users/RyanBard","html_url":"https://github.com/RyanBard","followers_url":"https://api.github.com/users/RyanBard/followers","following_url":"https://api.github.com/users/RyanBard/following{/other_user}","gists_url":"https://api.github.com/users/RyanBard/gists{/gist_id}","starred_url":"https://api.github.com/users/RyanBard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RyanBard/subscriptions","organizations_url":"https://api.github.com/users/RyanBard/orgs","repos_url":"https://api.github.com/users/RyanBard/repos","events_url":"https://api.github.com/users/RyanBard/events{/privacy}","received_events_url":"https://api.github.com/users/RyanBard/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/RyanBard/healthbam","description":"Healthy Mothers, Healthy Babies Coalition of Georgia Programs Map","fork":true,"url":"https://api.github.com/repos/RyanBard/healthbam","forks_url":"https://api.github.com/repos/RyanBard/healthbam/forks","keys_url":"https://api.github.com/repos/RyanBard/healthbam/keys{/key_id}","collaborators_url":"https://api.github.com/repos/RyanBard/healthbam/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/RyanBard/healthbam/teams","hooks_url":"https://api.github.com/repos/RyanBard/healthbam/hooks","issue_events_url":"https://api.github.com/repos/RyanBard/healthbam/issues/events{/number}","events_url":"https://api.github.com/repos/RyanBard/healthbam/events","assignees_url":"https://api.github.com/repos/RyanBard/healthbam/assignees{/user}","branches_url":"https://api.github.com/repos/RyanBard/healthbam/branches{/branch}","tags_url":"https://api.github.com/repos/RyanBard/healthbam/tags","blobs_url":"https://api.github.com/repos/RyanBard/healthbam/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/RyanBard/healthbam/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/RyanBard/healthbam/git/refs{/sha}","trees_url":"https://api.github.com/repos/RyanBard/healthbam/git/trees{/sha}","statuses_url":"https://api.github.com/repos/RyanBard/healthbam/statuses/{sha}","languages_url":"https://api.github.com/repos/RyanBard/healthbam/languages","stargazers_url":"https://api.github.com/repos/RyanBard/healthbam/stargazers","contributors_url":"https://api.github.com/repos/RyanBard/healthbam/contributors","subscribers_url":"https://api.github.com/repos/RyanBard/healthbam/subscribers","subscription_url":"https://api.github.com/repos/RyanBard/healthbam/subscription","commits_url":"https://api.github.com/repos/RyanBard/healthbam/commits{/sha}","git_commits_url":"https://api.github.com/repos/RyanBard/healthbam/git/commits{/sha}","comments_url":"https://api.github.com/repos/RyanBard/healthbam/comments{/number}","issue_comment_url":"https://api.github.com/repos/RyanBard/healthbam/issues/comments{/number}","contents_url":"https://api.github.com/repos/RyanBard/healthbam/contents/{+path}","compare_url":"https://api.github.com/repos/RyanBard/healthbam/compare/{base}...{head}","merges_url":"https://api.github.com/repos/RyanBard/healthbam/merges","archive_url":"https://api.github.com/repos/RyanBard/healthbam/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/RyanBard/healthbam/downloads","issues_url":"https://api.github.com/repos/RyanBard/healthbam/issues{/number}","pulls_url":"https://api.github.com/repos/RyanBard/healthbam/pulls{/number}","milestones_url":"https://api.github.com/repos/RyanBard/healthbam/milestones{/number}","notifications_url":"https://api.github.com/repos/RyanBard/healthbam/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/RyanBard/healthbam/labels{/name}","releases_url":"https://api.github.com/repos/RyanBard/healthbam/releases{/id}","deployments_url":"https://api.github.com/repos/RyanBard/healthbam/deployments","created_at":"2016-09-11T16:26:06Z","updated_at":"2016-10-12T01:55:39Z","pushed_at":"2016-10-30T01:51:31Z","git_url":"git://github.com/RyanBard/healthbam.git","ssh_url":"git@github.com:RyanBard/healthbam.git","clone_url":"https://github.com/RyanBard/healthbam.git","svn_url":"https://github.com/RyanBard/healthbam","homepage":null,"size":596,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"healthbam:master","ref":"master","sha":"66e9b9e57f792c00ef2c77277d1d7659783da319","user":{"login":"healthbam","id":21698178,"avatar_url":"https://avatars.githubusercontent.com/u/21698178?v=3","gravatar_id":"","url":"https://api.github.com/users/healthbam","html_url":"https://github.com/healthbam","followers_url":"https://api.github.com/users/healthbam/followers","following_url":"https://api.github.com/users/healthbam/following{/other_user}","gists_url":"https://api.github.com/users/healthbam/gists{/gist_id}","starred_url":"https://api.github.com/users/healthbam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/healthbam/subscriptions","organizations_url":"https://api.github.com/users/healthbam/orgs","repos_url":"https://api.github.com/users/healthbam/repos","events_url":"https://api.github.com/users/healthbam/events{/privacy}","received_events_url":"https://api.github.com/users/healthbam/received_events","type":"Organization","site_admin":false},"repo":{"id":67935826,"name":"healthbam","full_name":"healthbam/healthbam","owner":{"login":"healthbam","id":21698178,"avatar_url":"https://avatars.githubusercontent.com/u/21698178?v=3","gravatar_id":"","url":"https://api.github.com/users/healthbam","html_url":"https://github.com/healthbam","followers_url":"https://api.github.com/users/healthbam/followers","following_url":"https://api.github.com/users/healthbam/following{/other_user}","gists_url":"https://api.github.com/users/healthbam/gists{/gist_id}","starred_url":"https://api.github.com/users/healthbam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/healthbam/subscriptions","organizations_url":"https://api.github.com/users/healthbam/orgs","repos_url":"https://api.github.com/users/healthbam/repos","events_url":"https://api.github.com/users/healthbam/events{/privacy}","received_events_url":"https://api.github.com/users/healthbam/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/healthbam/healthbam","description":"Healthy Mothers, Healthy Babies Coalition of Georgia Programs Map","fork":false,"url":"https://api.github.com/repos/healthbam/healthbam","forks_url":"https://api.github.com/repos/healthbam/healthbam/forks","keys_url":"https://api.github.com/repos/healthbam/healthbam/keys{/key_id}","collaborators_url":"https://api.github.com/repos/healthbam/healthbam/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/healthbam/healthbam/teams","hooks_url":"https://api.github.com/repos/healthbam/healthbam/hooks","issue_events_url":"https://api.github.com/repos/healthbam/healthbam/issues/events{/number}","events_url":"https://api.github.com/repos/healthbam/healthbam/events","assignees_url":"https://api.github.com/repos/healthbam/healthbam/assignees{/user}","branches_url":"https://api.github.com/repos/healthbam/healthbam/branches{/branch}","tags_url":"https://api.github.com/repos/healthbam/healthbam/tags","blobs_url":"https://api.github.com/repos/healthbam/healthbam/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/healthbam/healthbam/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/healthbam/healthbam/git/refs{/sha}","trees_url":"https://api.github.com/repos/healthbam/healthbam/git/trees{/sha}","statuses_url":"https://api.github.com/repos/healthbam/healthbam/statuses/{sha}","languages_url":"https://api.github.com/repos/healthbam/healthbam/languages","stargazers_url":"https://api.github.com/repos/healthbam/healthbam/stargazers","contributors_url":"https://api.github.com/repos/healthbam/healthbam/contributors","subscribers_url":"https://api.github.com/repos/healthbam/healthbam/subscribers","subscription_url":"https://api.github.com/repos/healthbam/healthbam/subscription","commits_url":"https://api.github.com/repos/healthbam/healthbam/commits{/sha}","git_commits_url":"https://api.github.com/repos/healthbam/healthbam/git/commits{/sha}","comments_url":"https://api.github.com/repos/healthbam/healthbam/comments{/number}","issue_comment_url":"https://api.github.com/repos/healthbam/healthbam/issues/comments{/number}","contents_url":"https://api.github.com/repos/healthbam/healthbam/contents/{+path}","compare_url":"https://api.github.com/repos/healthbam/healthbam/compare/{base}...{head}","merges_url":"https://api.github.com/repos/healthbam/healthbam/merges","archive_url":"https://api.github.com/repos/healthbam/healthbam/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/healthbam/healthbam/downloads","issues_url":"https://api.github.com/repos/healthbam/healthbam/issues{/number}","pulls_url":"https://api.github.com/repos/healthbam/healthbam/pulls{/number}","milestones_url":"https://api.github.com/repos/healthbam/healthbam/milestones{/number}","notifications_url":"https://api.github.com/repos/healthbam/healthbam/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/healthbam/healthbam/labels{/name}","releases_url":"https://api.github.com/repos/healthbam/healthbam/releases{/id}","deployments_url":"https://api.github.com/repos/healthbam/healthbam/deployments","created_at":"2016-09-11T14:56:30Z","updated_at":"2016-10-16T02:41:20Z","pushed_at":"2016-10-30T01:52:12Z","git_url":"git://github.com/healthbam/healthbam.git","ssh_url":"git@github.com:healthbam/healthbam.git","clone_url":"https://github.com/healthbam/healthbam.git","svn_url":"https://github.com/healthbam/healthbam","homepage":null,"size":552,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":5,"mirror_url":null,"open_issues_count":34,"forks":5,"open_issues":34,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/healthbam/healthbam/pulls/233"},"html":{"href":"https://github.com/healthbam/healthbam/pull/233"},"issue":{"href":"https://api.github.com/repos/healthbam/healthbam/issues/233"},"comments":{"href":"https://api.github.com/repos/healthbam/healthbam/issues/233/comments"},"review_comments":{"href":"https://api.github.com/repos/healthbam/healthbam/pulls/233/comments"},"review_comment":{"href":"https://api.github.com/repos/healthbam/healthbam/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/healthbam/healthbam/pulls/233/commits"},"statuses":{"href":"https://api.github.com/repos/healthbam/healthbam/statuses/4aff58455d3606d44f7e1cdff7612c59effdbb7e"}}}}
{"actor":{"display_login":"RyanBard"}}