{"action":"created","comment":{"url":"https://api.github.com/repos/lookout/borderpatrol/pulls/comments/57774361","id":57774361,"diff_hunk":"@@ -73,12 +74,12 @@ object Keymaster {\n         case Status.Forbidden => {\n           statResponseDenied.incr\n           Future.exception(BpIdentityProviderError(Status.Forbidden,\n-            s\"IdentityProvider denied user: '${req.credential.uniqueId}' with status: ${res.status}\"))\n+            s\"IdentityProvider failed to authenticate user: '${req.credential.uniqueId}'\"))","path":"auth/src/main/scala/com/lookout/borderpatrol/auth/keymaster/Keymaster.scala","position":13,"original_position":13,"commit_id":"cc7b4b165aa94ce47232ab3e50599ee085c825e0","original_commit_id":"cc7b4b165aa94ce47232ab3e50599ee085c825e0","user":{"login":"maheshkelkar","id":10839177,"avatar_url":"https://avatars.githubusercontent.com/u/10839177?v=3","gravatar_id":"","url":"https://api.github.com/users/maheshkelkar","html_url":"https://github.com/maheshkelkar","followers_url":"https://api.github.com/users/maheshkelkar/followers","following_url":"https://api.github.com/users/maheshkelkar/following{/other_user}","gists_url":"https://api.github.com/users/maheshkelkar/gists{/gist_id}","starred_url":"https://api.github.com/users/maheshkelkar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maheshkelkar/subscriptions","organizations_url":"https://api.github.com/users/maheshkelkar/orgs","repos_url":"https://api.github.com/users/maheshkelkar/repos","events_url":"https://api.github.com/users/maheshkelkar/events{/privacy}","received_events_url":"https://api.github.com/users/maheshkelkar/received_events","type":"User","site_admin":false},"body":"Hi Peter, it indeed contains a username. We have a couple of more log messages that dump sensitive information such as token or username, x-forwarded-for. I will take them off as soon as we are done testing in the staging. ","created_at":"2016-03-29T18:25:41Z","updated_at":"2016-03-29T18:25:41Z","html_url":"https://github.com/lookout/borderpatrol/pull/137#discussion_r57774361","pull_request_url":"https://api.github.com/repos/lookout/borderpatrol/pulls/137","_links":{"self":{"href":"https://api.github.com/repos/lookout/borderpatrol/pulls/comments/57774361"},"html":{"href":"https://github.com/lookout/borderpatrol/pull/137#discussion_r57774361"},"pull_request":{"href":"https://api.github.com/repos/lookout/borderpatrol/pulls/137"}}},"pull_request":{"url":"https://api.github.com/repos/lookout/borderpatrol/pulls/137","id":64508794,"html_url":"https://github.com/lookout/borderpatrol/pull/137","diff_url":"https://github.com/lookout/borderpatrol/pull/137.diff","patch_url":"https://github.com/lookout/borderpatrol/pull/137.patch","issue_url":"https://api.github.com/repos/lookout/borderpatrol/issues/137","number":137,"state":"open","locked":false,"title":"Updated docs and minor fixes","user":{"login":"maheshkelkar","id":10839177,"avatar_url":"https://avatars.githubusercontent.com/u/10839177?v=3","gravatar_id":"","url":"https://api.github.com/users/maheshkelkar","html_url":"https://github.com/maheshkelkar","followers_url":"https://api.github.com/users/maheshkelkar/followers","following_url":"https://api.github.com/users/maheshkelkar/following{/other_user}","gists_url":"https://api.github.com/users/maheshkelkar/gists{/gist_id}","starred_url":"https://api.github.com/users/maheshkelkar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maheshkelkar/subscriptions","organizations_url":"https://api.github.com/users/maheshkelkar/orgs","repos_url":"https://api.github.com/users/maheshkelkar/repos","events_url":"https://api.github.com/users/maheshkelkar/events{/privacy}","received_events_url":"https://api.github.com/users/maheshkelkar/received_events","type":"User","site_admin":false},"body":"- Updated README.md with Block Diagram, Concepts, Components,\r\n  Configuration, etc\r\n- Updated contents of some error logs\r\n- Added code to scrub special characters from query parameters\r\n- Added logs to dump X-Forwarded-* for debugging\r\n- Fix the version in HealthCheck output","created_at":"2016-03-29T18:11:21Z","updated_at":"2016-03-29T18:25:41Z","closed_at":null,"merged_at":null,"merge_commit_sha":"3b75f6f66f2a8963ea0e1453ad7dcc6e77cac352","assignee":{"login":"PeterFCaswell","id":11891688,"avatar_url":"https://avatars.githubusercontent.com/u/11891688?v=3","gravatar_id":"","url":"https://api.github.com/users/PeterFCaswell","html_url":"https://github.com/PeterFCaswell","followers_url":"https://api.github.com/users/PeterFCaswell/followers","following_url":"https://api.github.com/users/PeterFCaswell/following{/other_user}","gists_url":"https://api.github.com/users/PeterFCaswell/gists{/gist_id}","starred_url":"https://api.github.com/users/PeterFCaswell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PeterFCaswell/subscriptions","organizations_url":"https://api.github.com/users/PeterFCaswell/orgs","repos_url":"https://api.github.com/users/PeterFCaswell/repos","events_url":"https://api.github.com/users/PeterFCaswell/events{/privacy}","received_events_url":"https://api.github.com/users/PeterFCaswell/received_events","type":"User","site_admin":false},"milestone":null,"commits_url":"https://api.github.com/repos/lookout/borderpatrol/pulls/137/commits","review_comments_url":"https://api.github.com/repos/lookout/borderpatrol/pulls/137/comments","review_comment_url":"https://api.github.com/repos/lookout/borderpatrol/pulls/comments{/number}","comments_url":"https://api.github.com/repos/lookout/borderpatrol/issues/137/comments","statuses_url":"https://api.github.com/repos/lookout/borderpatrol/statuses/cc7b4b165aa94ce47232ab3e50599ee085c825e0","head":{"label":"maheshkelkar:issue-update-docs-and-fixes","ref":"issue-update-docs-and-fixes","sha":"cc7b4b165aa94ce47232ab3e50599ee085c825e0","user":{"login":"maheshkelkar","id":10839177,"avatar_url":"https://avatars.githubusercontent.com/u/10839177?v=3","gravatar_id":"","url":"https://api.github.com/users/maheshkelkar","html_url":"https://github.com/maheshkelkar","followers_url":"https://api.github.com/users/maheshkelkar/followers","following_url":"https://api.github.com/users/maheshkelkar/following{/other_user}","gists_url":"https://api.github.com/users/maheshkelkar/gists{/gist_id}","starred_url":"https://api.github.com/users/maheshkelkar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maheshkelkar/subscriptions","organizations_url":"https://api.github.com/users/maheshkelkar/orgs","repos_url":"https://api.github.com/users/maheshkelkar/repos","events_url":"https://api.github.com/users/maheshkelkar/events{/privacy}","received_events_url":"https://api.github.com/users/maheshkelkar/received_events","type":"User","site_admin":false},"repo":{"id":41160852,"name":"borderpatrol","full_name":"maheshkelkar/borderpatrol","owner":{"login":"maheshkelkar","id":10839177,"avatar_url":"https://avatars.githubusercontent.com/u/10839177?v=3","gravatar_id":"","url":"https://api.github.com/users/maheshkelkar","html_url":"https://github.com/maheshkelkar","followers_url":"https://api.github.com/users/maheshkelkar/followers","following_url":"https://api.github.com/users/maheshkelkar/following{/other_user}","gists_url":"https://api.github.com/users/maheshkelkar/gists{/gist_id}","starred_url":"https://api.github.com/users/maheshkelkar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maheshkelkar/subscriptions","organizations_url":"https://api.github.com/users/maheshkelkar/orgs","repos_url":"https://api.github.com/users/maheshkelkar/repos","events_url":"https://api.github.com/users/maheshkelkar/events{/privacy}","received_events_url":"https://api.github.com/users/maheshkelkar/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/maheshkelkar/borderpatrol","description":"Functional library for web sessions and authentication in an SOA","fork":true,"url":"https://api.github.com/repos/maheshkelkar/borderpatrol","forks_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/forks","keys_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/keys{/key_id}","collaborators_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/teams","hooks_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/hooks","issue_events_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/issues/events{/number}","events_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/events","assignees_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/assignees{/user}","branches_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/branches{/branch}","tags_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/tags","blobs_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/git/refs{/sha}","trees_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/git/trees{/sha}","statuses_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/statuses/{sha}","languages_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/languages","stargazers_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/stargazers","contributors_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/contributors","subscribers_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/subscribers","subscription_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/subscription","commits_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/commits{/sha}","git_commits_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/git/commits{/sha}","comments_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/comments{/number}","issue_comment_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/issues/comments{/number}","contents_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/contents/{+path}","compare_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/compare/{base}...{head}","merges_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/merges","archive_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/downloads","issues_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/issues{/number}","pulls_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/pulls{/number}","milestones_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/milestones{/number}","notifications_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/labels{/name}","releases_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/releases{/id}","deployments_url":"https://api.github.com/repos/maheshkelkar/borderpatrol/deployments","created_at":"2015-08-21T14:40:33Z","updated_at":"2015-08-21T14:40:33Z","pushed_at":"2016-03-29T18:10:11Z","git_url":"git://github.com/maheshkelkar/borderpatrol.git","ssh_url":"git@github.com:maheshkelkar/borderpatrol.git","clone_url":"https://github.com/maheshkelkar/borderpatrol.git","svn_url":"https://github.com/maheshkelkar/borderpatrol","homepage":"http://hackers.lookout.com/2014/06/introducing-borderpatrol/","size":1486,"stargazers_count":0,"watchers_count":0,"language":"Scala","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":"lookout:master","ref":"master","sha":"45582feefde4e29c3558c023f3d51718632329eb","user":{"login":"lookout","id":47237,"avatar_url":"https://avatars.githubusercontent.com/u/47237?v=3","gravatar_id":"","url":"https://api.github.com/users/lookout","html_url":"https://github.com/lookout","followers_url":"https://api.github.com/users/lookout/followers","following_url":"https://api.github.com/users/lookout/following{/other_user}","gists_url":"https://api.github.com/users/lookout/gists{/gist_id}","starred_url":"https://api.github.com/users/lookout/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lookout/subscriptions","organizations_url":"https://api.github.com/users/lookout/orgs","repos_url":"https://api.github.com/users/lookout/repos","events_url":"https://api.github.com/users/lookout/events{/privacy}","received_events_url":"https://api.github.com/users/lookout/received_events","type":"Organization","site_admin":false},"repo":{"id":20500575,"name":"borderpatrol","full_name":"lookout/borderpatrol","owner":{"login":"lookout","id":47237,"avatar_url":"https://avatars.githubusercontent.com/u/47237?v=3","gravatar_id":"","url":"https://api.github.com/users/lookout","html_url":"https://github.com/lookout","followers_url":"https://api.github.com/users/lookout/followers","following_url":"https://api.github.com/users/lookout/following{/other_user}","gists_url":"https://api.github.com/users/lookout/gists{/gist_id}","starred_url":"https://api.github.com/users/lookout/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lookout/subscriptions","organizations_url":"https://api.github.com/users/lookout/orgs","repos_url":"https://api.github.com/users/lookout/repos","events_url":"https://api.github.com/users/lookout/events{/privacy}","received_events_url":"https://api.github.com/users/lookout/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/lookout/borderpatrol","description":"Functional library for web sessions and authentication in an SOA","fork":false,"url":"https://api.github.com/repos/lookout/borderpatrol","forks_url":"https://api.github.com/repos/lookout/borderpatrol/forks","keys_url":"https://api.github.com/repos/lookout/borderpatrol/keys{/key_id}","collaborators_url":"https://api.github.com/repos/lookout/borderpatrol/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/lookout/borderpatrol/teams","hooks_url":"https://api.github.com/repos/lookout/borderpatrol/hooks","issue_events_url":"https://api.github.com/repos/lookout/borderpatrol/issues/events{/number}","events_url":"https://api.github.com/repos/lookout/borderpatrol/events","assignees_url":"https://api.github.com/repos/lookout/borderpatrol/assignees{/user}","branches_url":"https://api.github.com/repos/lookout/borderpatrol/branches{/branch}","tags_url":"https://api.github.com/repos/lookout/borderpatrol/tags","blobs_url":"https://api.github.com/repos/lookout/borderpatrol/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/lookout/borderpatrol/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/lookout/borderpatrol/git/refs{/sha}","trees_url":"https://api.github.com/repos/lookout/borderpatrol/git/trees{/sha}","statuses_url":"https://api.github.com/repos/lookout/borderpatrol/statuses/{sha}","languages_url":"https://api.github.com/repos/lookout/borderpatrol/languages","stargazers_url":"https://api.github.com/repos/lookout/borderpatrol/stargazers","contributors_url":"https://api.github.com/repos/lookout/borderpatrol/contributors","subscribers_url":"https://api.github.com/repos/lookout/borderpatrol/subscribers","subscription_url":"https://api.github.com/repos/lookout/borderpatrol/subscription","commits_url":"https://api.github.com/repos/lookout/borderpatrol/commits{/sha}","git_commits_url":"https://api.github.com/repos/lookout/borderpatrol/git/commits{/sha}","comments_url":"https://api.github.com/repos/lookout/borderpatrol/comments{/number}","issue_comment_url":"https://api.github.com/repos/lookout/borderpatrol/issues/comments{/number}","contents_url":"https://api.github.com/repos/lookout/borderpatrol/contents/{+path}","compare_url":"https://api.github.com/repos/lookout/borderpatrol/compare/{base}...{head}","merges_url":"https://api.github.com/repos/lookout/borderpatrol/merges","archive_url":"https://api.github.com/repos/lookout/borderpatrol/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/lookout/borderpatrol/downloads","issues_url":"https://api.github.com/repos/lookout/borderpatrol/issues{/number}","pulls_url":"https://api.github.com/repos/lookout/borderpatrol/pulls{/number}","milestones_url":"https://api.github.com/repos/lookout/borderpatrol/milestones{/number}","notifications_url":"https://api.github.com/repos/lookout/borderpatrol/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/lookout/borderpatrol/labels{/name}","releases_url":"https://api.github.com/repos/lookout/borderpatrol/releases{/id}","deployments_url":"https://api.github.com/repos/lookout/borderpatrol/deployments","created_at":"2014-06-04T21:00:49Z","updated_at":"2016-03-29T18:25:29Z","pushed_at":"2016-03-29T18:11:21Z","git_url":"git://github.com/lookout/borderpatrol.git","ssh_url":"git@github.com:lookout/borderpatrol.git","clone_url":"https://github.com/lookout/borderpatrol.git","svn_url":"https://github.com/lookout/borderpatrol","homepage":"http://hackers.lookout.com/2014/06/introducing-borderpatrol/","size":1574,"stargazers_count":15,"watchers_count":15,"language":"Scala","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":13,"mirror_url":null,"open_issues_count":17,"forks":13,"open_issues":17,"watchers":15,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/lookout/borderpatrol/pulls/137"},"html":{"href":"https://github.com/lookout/borderpatrol/pull/137"},"issue":{"href":"https://api.github.com/repos/lookout/borderpatrol/issues/137"},"comments":{"href":"https://api.github.com/repos/lookout/borderpatrol/issues/137/comments"},"review_comments":{"href":"https://api.github.com/repos/lookout/borderpatrol/pulls/137/comments"},"review_comment":{"href":"https://api.github.com/repos/lookout/borderpatrol/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/lookout/borderpatrol/pulls/137/commits"},"statuses":{"href":"https://api.github.com/repos/lookout/borderpatrol/statuses/cc7b4b165aa94ce47232ab3e50599ee085c825e0"}}}}