GitHub PullRequestReviewCommentEvent at 2014-10-30 23:50:14
Repository: OpenBazaar
Actor: Renelvon
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/comments/19644555","id":19644555,"diff_hunk":"@@ -473,8 +480,11 @@ def _on_message(self, msg):\n         uri = msg.get('uri')\n         guid = msg.get('senderGUID')\n         nickname = msg.get('senderNick')[:120]\n+        msgType = msg.get('type')\n \n-        self.log.info('On Message: %s', json.dumps(msg, ensure_ascii=False))\n+        self.log.info(\"Received message type '%s' from \\\"%s\\\" %s %s\",","path":"node/transport.py","position":42,"original_position":42,"commit_id":"5f0d53285b1bc2ba72ac1feced9b5c976e000044","original_commit_id":"5f0d53285b1bc2ba72ac1feced9b5c976e000044","user":{"login":"Renelvon","id":860034,"avatar_url":"https://avatars.githubusercontent.com/u/860034?v=2","gravatar_id":"","url":"https://api.github.com/users/Renelvon","html_url":"https://github.com/Renelvon","followers_url":"https://api.github.com/users/Renelvon/followers","following_url":"https://api.github.com/users/Renelvon/following{/other_user}","gists_url":"https://api.github.com/users/Renelvon/gists{/gist_id}","starred_url":"https://api.github.com/users/Renelvon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Renelvon/subscriptions","organizations_url":"https://api.github.com/users/Renelvon/orgs","repos_url":"https://api.github.com/users/Renelvon/repos","events_url":"https://api.github.com/users/Renelvon/events{/privacy}","received_events_url":"https://api.github.com/users/Renelvon/received_events","type":"User","site_admin":false},"body":"Then:\r\n```python\r\n... 'Received message type \"%s\" from \"%s\" %s %s\"...\r\n```","created_at":"2014-10-30T23:50:14Z","updated_at":"2014-10-30T23:50:14Z","html_url":"https://github.com/OpenBazaar/OpenBazaar/pull/889#discussion_r19644555","pull_request_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/889","_links":{"self":{"href":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/comments/19644555"},"html":{"href":"https://github.com/OpenBazaar/OpenBazaar/pull/889#discussion_r19644555"},"pull_request":{"href":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/889"}}},"pull_request":{"url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/889","id":23633443,"html_url":"https://github.com/OpenBazaar/OpenBazaar/pull/889","diff_url":"https://github.com/OpenBazaar/OpenBazaar/pull/889.diff","patch_url":"https://github.com/OpenBazaar/OpenBazaar/pull/889.patch","issue_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/issues/889","number":889,"state":"open","locked":false,"title":"log output: initial move to DATADUMP","user":{"login":"benemorius","id":64743,"avatar_url":"https://avatars.githubusercontent.com/u/64743?v=2","gravatar_id":"","url":"https://api.github.com/users/benemorius","html_url":"https://github.com/benemorius","followers_url":"https://api.github.com/users/benemorius/followers","following_url":"https://api.github.com/users/benemorius/following{/other_user}","gists_url":"https://api.github.com/users/benemorius/gists{/gist_id}","starred_url":"https://api.github.com/users/benemorius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benemorius/subscriptions","organizations_url":"https://api.github.com/users/benemorius/orgs","repos_url":"https://api.github.com/users/benemorius/repos","events_url":"https://api.github.com/users/benemorius/events{/privacy}","received_events_url":"https://api.github.com/users/benemorius/received_events","type":"User","site_admin":false},"body":"This moves most of the biggest log outputs from DEBUG and INFO to DATADUMP. Turns out, there weren't all that many.\r\n\r\nNext I'm reevaluating the log level assigned to each output. Ideally I'm thinking the default log level should be INFO (20) and should give the user a good picture of what their node is doing on the network without being too much or too many to read in realtime.","created_at":"2014-10-30T19:53:21Z","updated_at":"2014-10-30T23:50:14Z","closed_at":null,"merged_at":null,"merge_commit_sha":"0913336681fd56a9ce86be1a628ae3ece209525c","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/889/commits","review_comments_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/889/comments","review_comment_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/comments/{number}","comments_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/issues/889/comments","statuses_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/statuses/5f0d53285b1bc2ba72ac1feced9b5c976e000044","head":{"label":"benemorius:log-datadump","ref":"log-datadump","sha":"5f0d53285b1bc2ba72ac1feced9b5c976e000044","user":{"login":"benemorius","id":64743,"avatar_url":"https://avatars.githubusercontent.com/u/64743?v=2","gravatar_id":"","url":"https://api.github.com/users/benemorius","html_url":"https://github.com/benemorius","followers_url":"https://api.github.com/users/benemorius/followers","following_url":"https://api.github.com/users/benemorius/following{/other_user}","gists_url":"https://api.github.com/users/benemorius/gists{/gist_id}","starred_url":"https://api.github.com/users/benemorius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benemorius/subscriptions","organizations_url":"https://api.github.com/users/benemorius/orgs","repos_url":"https://api.github.com/users/benemorius/repos","events_url":"https://api.github.com/users/benemorius/events{/privacy}","received_events_url":"https://api.github.com/users/benemorius/received_events","type":"User","site_admin":false},"repo":{"id":23536469,"name":"OpenBazaar","full_name":"benemorius/OpenBazaar","owner":{"login":"benemorius","id":64743,"avatar_url":"https://avatars.githubusercontent.com/u/64743?v=2","gravatar_id":"","url":"https://api.github.com/users/benemorius","html_url":"https://github.com/benemorius","followers_url":"https://api.github.com/users/benemorius/followers","following_url":"https://api.github.com/users/benemorius/following{/other_user}","gists_url":"https://api.github.com/users/benemorius/gists{/gist_id}","starred_url":"https://api.github.com/users/benemorius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benemorius/subscriptions","organizations_url":"https://api.github.com/users/benemorius/orgs","repos_url":"https://api.github.com/users/benemorius/repos","events_url":"https://api.github.com/users/benemorius/events{/privacy}","received_events_url":"https://api.github.com/users/benemorius/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/benemorius/OpenBazaar","description":"A decentralized marketplace","fork":true,"url":"https://api.github.com/repos/benemorius/OpenBazaar","forks_url":"https://api.github.com/repos/benemorius/OpenBazaar/forks","keys_url":"https://api.github.com/repos/benemorius/OpenBazaar/keys{/key_id}","collaborators_url":"https://api.github.com/repos/benemorius/OpenBazaar/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/benemorius/OpenBazaar/teams","hooks_url":"https://api.github.com/repos/benemorius/OpenBazaar/hooks","issue_events_url":"https://api.github.com/repos/benemorius/OpenBazaar/issues/events{/number}","events_url":"https://api.github.com/repos/benemorius/OpenBazaar/events","assignees_url":"https://api.github.com/repos/benemorius/OpenBazaar/assignees{/user}","branches_url":"https://api.github.com/repos/benemorius/OpenBazaar/branches{/branch}","tags_url":"https://api.github.com/repos/benemorius/OpenBazaar/tags","blobs_url":"https://api.github.com/repos/benemorius/OpenBazaar/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/benemorius/OpenBazaar/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/benemorius/OpenBazaar/git/refs{/sha}","trees_url":"https://api.github.com/repos/benemorius/OpenBazaar/git/trees{/sha}","statuses_url":"https://api.github.com/repos/benemorius/OpenBazaar/statuses/{sha}","languages_url":"https://api.github.com/repos/benemorius/OpenBazaar/languages","stargazers_url":"https://api.github.com/repos/benemorius/OpenBazaar/stargazers","contributors_url":"https://api.github.com/repos/benemorius/OpenBazaar/contributors","subscribers_url":"https://api.github.com/repos/benemorius/OpenBazaar/subscribers","subscription_url":"https://api.github.com/repos/benemorius/OpenBazaar/subscription","commits_url":"https://api.github.com/repos/benemorius/OpenBazaar/commits{/sha}","git_commits_url":"https://api.github.com/repos/benemorius/OpenBazaar/git/commits{/sha}","comments_url":"https://api.github.com/repos/benemorius/OpenBazaar/comments{/number}","issue_comment_url":"https://api.github.com/repos/benemorius/OpenBazaar/issues/comments/{number}","contents_url":"https://api.github.com/repos/benemorius/OpenBazaar/contents/{+path}","compare_url":"https://api.github.com/repos/benemorius/OpenBazaar/compare/{base}...{head}","merges_url":"https://api.github.com/repos/benemorius/OpenBazaar/merges","archive_url":"https://api.github.com/repos/benemorius/OpenBazaar/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/benemorius/OpenBazaar/downloads","issues_url":"https://api.github.com/repos/benemorius/OpenBazaar/issues{/number}","pulls_url":"https://api.github.com/repos/benemorius/OpenBazaar/pulls{/number}","milestones_url":"https://api.github.com/repos/benemorius/OpenBazaar/milestones{/number}","notifications_url":"https://api.github.com/repos/benemorius/OpenBazaar/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/benemorius/OpenBazaar/labels{/name}","releases_url":"https://api.github.com/repos/benemorius/OpenBazaar/releases{/id}","created_at":"2014-09-01T08:54:38Z","updated_at":"2014-09-02T03:11:49Z","pushed_at":"2014-10-30T22:04:21Z","git_url":"git://github.com/benemorius/OpenBazaar.git","ssh_url":"git@github.com:benemorius/OpenBazaar.git","clone_url":"https://github.com/benemorius/OpenBazaar.git","svn_url":"https://github.com/benemorius/OpenBazaar","homepage":"http://forum.openbazaar.org","size":21027,"stargazers_count":0,"watchers_count":0,"language":"Python","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":"dev"}},"base":{"label":"OpenBazaar:develop","ref":"develop","sha":"e026aafb4264cdf708c2daf10fa77ae6ea85b805","user":{"login":"OpenBazaar","id":7438770,"avatar_url":"https://avatars.githubusercontent.com/u/7438770?v=2","gravatar_id":"","url":"https://api.github.com/users/OpenBazaar","html_url":"https://github.com/OpenBazaar","followers_url":"https://api.github.com/users/OpenBazaar/followers","following_url":"https://api.github.com/users/OpenBazaar/following{/other_user}","gists_url":"https://api.github.com/users/OpenBazaar/gists{/gist_id}","starred_url":"https://api.github.com/users/OpenBazaar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OpenBazaar/subscriptions","organizations_url":"https://api.github.com/users/OpenBazaar/orgs","repos_url":"https://api.github.com/users/OpenBazaar/repos","events_url":"https://api.github.com/users/OpenBazaar/events{/privacy}","received_events_url":"https://api.github.com/users/OpenBazaar/received_events","type":"Organization","site_admin":false},"repo":{"id":19147609,"name":"OpenBazaar","full_name":"OpenBazaar/OpenBazaar","owner":{"login":"OpenBazaar","id":7438770,"avatar_url":"https://avatars.githubusercontent.com/u/7438770?v=2","gravatar_id":"","url":"https://api.github.com/users/OpenBazaar","html_url":"https://github.com/OpenBazaar","followers_url":"https://api.github.com/users/OpenBazaar/followers","following_url":"https://api.github.com/users/OpenBazaar/following{/other_user}","gists_url":"https://api.github.com/users/OpenBazaar/gists{/gist_id}","starred_url":"https://api.github.com/users/OpenBazaar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OpenBazaar/subscriptions","organizations_url":"https://api.github.com/users/OpenBazaar/orgs","repos_url":"https://api.github.com/users/OpenBazaar/repos","events_url":"https://api.github.com/users/OpenBazaar/events{/privacy}","received_events_url":"https://api.github.com/users/OpenBazaar/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/OpenBazaar/OpenBazaar","description":"A decentralized marketplace","fork":false,"url":"https://api.github.com/repos/OpenBazaar/OpenBazaar","forks_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/forks","keys_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/keys{/key_id}","collaborators_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/teams","hooks_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/hooks","issue_events_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/issues/events{/number}","events_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/events","assignees_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/assignees{/user}","branches_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/branches{/branch}","tags_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/tags","blobs_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/git/refs{/sha}","trees_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/git/trees{/sha}","statuses_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/statuses/{sha}","languages_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/languages","stargazers_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/stargazers","contributors_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/contributors","subscribers_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/subscribers","subscription_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/subscription","commits_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/commits{/sha}","git_commits_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/git/commits{/sha}","comments_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/comments{/number}","issue_comment_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/issues/comments/{number}","contents_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/contents/{+path}","compare_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/compare/{base}...{head}","merges_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/merges","archive_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/downloads","issues_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/issues{/number}","pulls_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls{/number}","milestones_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/milestones{/number}","notifications_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/labels{/name}","releases_url":"https://api.github.com/repos/OpenBazaar/OpenBazaar/releases{/id}","created_at":"2014-04-25T13:49:36Z","updated_at":"2014-10-30T22:19:26Z","pushed_at":"2014-10-30T16:25:12Z","git_url":"git://github.com/OpenBazaar/OpenBazaar.git","ssh_url":"git@github.com:OpenBazaar/OpenBazaar.git","clone_url":"https://github.com/OpenBazaar/OpenBazaar.git","svn_url":"https://github.com/OpenBazaar/OpenBazaar","homepage":"http://forum.openbazaar.org","size":39452,"stargazers_count":726,"watchers_count":726,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":211,"mirror_url":null,"open_issues_count":181,"forks":211,"open_issues":181,"watchers":726,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/889"},"html":{"href":"https://github.com/OpenBazaar/OpenBazaar/pull/889"},"issue":{"href":"https://api.github.com/repos/OpenBazaar/OpenBazaar/issues/889"},"comments":{"href":"https://api.github.com/repos/OpenBazaar/OpenBazaar/issues/889/comments"},"review_comments":{"href":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/889/comments"},"review_comment":{"href":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/OpenBazaar/OpenBazaar/pulls/889/commits"},"statuses":{"href":"https://api.github.com/repos/OpenBazaar/OpenBazaar/statuses/5f0d53285b1bc2ba72ac1feced9b5c976e000044"}}}}