{"action":"created","comment":{"url":"https://api.github.com/repos/maidsafe/routing/pulls/comments/35304973","id":35304973,"diff_hunk":"@@ -557,7 +561,12 @@ impl<F> RoutingMembrane<F> where F: Interface {\n                         message.clone(), request.clone()),\n                     MessageType::GetDataResponse(ref response) =>\n                         self.handle_get_data_response(message_wrap, message.clone(), response.clone()),\n-                    MessageType::PutData(ref data) => self.handle_put_data(message_wrap, message.clone(), data.clone()),\n+                    MessageType::PutData(ref data) => {\n+                        match message.source.actual_source() {\n+                            Address::Node(name) => self.handle_group_put_data(message_wrap, message.clone(), data.clone(), name),\n+                            Address::Client(name) => self.handle_client_put_data(message_wrap, message.clone(), data.clone(), name),\n+                        }\n+                    },","path":"src/routing_membrane.rs","position":59,"original_position":59,"commit_id":"16b2386f7176983c7edd7d767c3403a190e226ee","original_commit_id":"16b2386f7176983c7edd7d767c3403a190e226ee","user":{"login":"benjaminbollen","id":5057507,"avatar_url":"https://avatars.githubusercontent.com/u/5057507?v=3","gravatar_id":"","url":"https://api.github.com/users/benjaminbollen","html_url":"https://github.com/benjaminbollen","followers_url":"https://api.github.com/users/benjaminbollen/followers","following_url":"https://api.github.com/users/benjaminbollen/following{/other_user}","gists_url":"https://api.github.com/users/benjaminbollen/gists{/gist_id}","starred_url":"https://api.github.com/users/benjaminbollen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benjaminbollen/subscriptions","organizations_url":"https://api.github.com/users/benjaminbollen/orgs","repos_url":"https://api.github.com/users/benjaminbollen/repos","events_url":"https://api.github.com/users/benjaminbollen/events{/privacy}","received_events_url":"https://api.github.com/users/benjaminbollen/received_events","type":"User","site_admin":false},"body":"mahmoud, please use message.from_group() as this is the way the sender can and will indicate how it thinks we should validate its message.","created_at":"2015-07-23T09:38:21Z","updated_at":"2015-07-23T09:38:21Z","html_url":"https://github.com/maidsafe/routing/pull/499#discussion_r35304973","pull_request_url":"https://api.github.com/repos/maidsafe/routing/pulls/499","_links":{"self":{"href":"https://api.github.com/repos/maidsafe/routing/pulls/comments/35304973"},"html":{"href":"https://github.com/maidsafe/routing/pull/499#discussion_r35304973"},"pull_request":{"href":"https://api.github.com/repos/maidsafe/routing/pulls/499"}}},"pull_request":{"url":"https://api.github.com/repos/maidsafe/routing/pulls/499","id":40476148,"html_url":"https://github.com/maidsafe/routing/pull/499","diff_url":"https://github.com/maidsafe/routing/pull/499.diff","patch_url":"https://github.com/maidsafe/routing/pull/499.patch","issue_url":"https://api.github.com/repos/maidsafe/routing/issues/499","number":499,"state":"open","locked":false,"title":"Refactor RoutingMembrane::handle_put_data as preparation for adding PureSentinel: MAID-1164","user":{"login":"mmoadeli","id":540171,"avatar_url":"https://avatars.githubusercontent.com/u/540171?v=3","gravatar_id":"","url":"https://api.github.com/users/mmoadeli","html_url":"https://github.com/mmoadeli","followers_url":"https://api.github.com/users/mmoadeli/followers","following_url":"https://api.github.com/users/mmoadeli/following{/other_user}","gists_url":"https://api.github.com/users/mmoadeli/gists{/gist_id}","starred_url":"https://api.github.com/users/mmoadeli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmoadeli/subscriptions","organizations_url":"https://api.github.com/users/mmoadeli/orgs","repos_url":"https://api.github.com/users/mmoadeli/repos","events_url":"https://api.github.com/users/mmoadeli/events{/privacy}","received_events_url":"https://api.github.com/users/mmoadeli/received_events","type":"User","site_admin":false},"body":"\n\n<!-- Reviewable:start -->\n[<img src=\"https://reviewable.io/review_button.png\" height=40 alt=\"Review on Reviewable\"/>](https://reviewable.io/reviews/maidsafe/routing/499)\n<!-- Reviewable:end -->\n","created_at":"2015-07-21T16:20:47Z","updated_at":"2015-07-23T09:38:21Z","closed_at":null,"merged_at":null,"merge_commit_sha":"35616681b5de22350ef2b4a7eb06b07c6e62c26d","assignee":{"login":"benjaminbollen","id":5057507,"avatar_url":"https://avatars.githubusercontent.com/u/5057507?v=3","gravatar_id":"","url":"https://api.github.com/users/benjaminbollen","html_url":"https://github.com/benjaminbollen","followers_url":"https://api.github.com/users/benjaminbollen/followers","following_url":"https://api.github.com/users/benjaminbollen/following{/other_user}","gists_url":"https://api.github.com/users/benjaminbollen/gists{/gist_id}","starred_url":"https://api.github.com/users/benjaminbollen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benjaminbollen/subscriptions","organizations_url":"https://api.github.com/users/benjaminbollen/orgs","repos_url":"https://api.github.com/users/benjaminbollen/repos","events_url":"https://api.github.com/users/benjaminbollen/events{/privacy}","received_events_url":"https://api.github.com/users/benjaminbollen/received_events","type":"User","site_admin":false},"milestone":null,"commits_url":"https://api.github.com/repos/maidsafe/routing/pulls/499/commits","review_comments_url":"https://api.github.com/repos/maidsafe/routing/pulls/499/comments","review_comment_url":"https://api.github.com/repos/maidsafe/routing/pulls/comments{/number}","comments_url":"https://api.github.com/repos/maidsafe/routing/issues/499/comments","statuses_url":"https://api.github.com/repos/maidsafe/routing/statuses/16b2386f7176983c7edd7d767c3403a190e226ee","head":{"label":"mmoadeli:fix","ref":"fix","sha":"16b2386f7176983c7edd7d767c3403a190e226ee","user":{"login":"mmoadeli","id":540171,"avatar_url":"https://avatars.githubusercontent.com/u/540171?v=3","gravatar_id":"","url":"https://api.github.com/users/mmoadeli","html_url":"https://github.com/mmoadeli","followers_url":"https://api.github.com/users/mmoadeli/followers","following_url":"https://api.github.com/users/mmoadeli/following{/other_user}","gists_url":"https://api.github.com/users/mmoadeli/gists{/gist_id}","starred_url":"https://api.github.com/users/mmoadeli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmoadeli/subscriptions","organizations_url":"https://api.github.com/users/mmoadeli/orgs","repos_url":"https://api.github.com/users/mmoadeli/repos","events_url":"https://api.github.com/users/mmoadeli/events{/privacy}","received_events_url":"https://api.github.com/users/mmoadeli/received_events","type":"User","site_admin":false},"repo":{"id":34522113,"name":"routing","full_name":"mmoadeli/routing","owner":{"login":"mmoadeli","id":540171,"avatar_url":"https://avatars.githubusercontent.com/u/540171?v=3","gravatar_id":"","url":"https://api.github.com/users/mmoadeli","html_url":"https://github.com/mmoadeli","followers_url":"https://api.github.com/users/mmoadeli/followers","following_url":"https://api.github.com/users/mmoadeli/following{/other_user}","gists_url":"https://api.github.com/users/mmoadeli/gists{/gist_id}","starred_url":"https://api.github.com/users/mmoadeli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmoadeli/subscriptions","organizations_url":"https://api.github.com/users/mmoadeli/orgs","repos_url":"https://api.github.com/users/mmoadeli/repos","events_url":"https://api.github.com/users/mmoadeli/events{/privacy}","received_events_url":"https://api.github.com/users/mmoadeli/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/mmoadeli/routing","description":"Routing - specialised storage DHT","fork":true,"url":"https://api.github.com/repos/mmoadeli/routing","forks_url":"https://api.github.com/repos/mmoadeli/routing/forks","keys_url":"https://api.github.com/repos/mmoadeli/routing/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mmoadeli/routing/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mmoadeli/routing/teams","hooks_url":"https://api.github.com/repos/mmoadeli/routing/hooks","issue_events_url":"https://api.github.com/repos/mmoadeli/routing/issues/events{/number}","events_url":"https://api.github.com/repos/mmoadeli/routing/events","assignees_url":"https://api.github.com/repos/mmoadeli/routing/assignees{/user}","branches_url":"https://api.github.com/repos/mmoadeli/routing/branches{/branch}","tags_url":"https://api.github.com/repos/mmoadeli/routing/tags","blobs_url":"https://api.github.com/repos/mmoadeli/routing/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mmoadeli/routing/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mmoadeli/routing/git/refs{/sha}","trees_url":"https://api.github.com/repos/mmoadeli/routing/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mmoadeli/routing/statuses/{sha}","languages_url":"https://api.github.com/repos/mmoadeli/routing/languages","stargazers_url":"https://api.github.com/repos/mmoadeli/routing/stargazers","contributors_url":"https://api.github.com/repos/mmoadeli/routing/contributors","subscribers_url":"https://api.github.com/repos/mmoadeli/routing/subscribers","subscription_url":"https://api.github.com/repos/mmoadeli/routing/subscription","commits_url":"https://api.github.com/repos/mmoadeli/routing/commits{/sha}","git_commits_url":"https://api.github.com/repos/mmoadeli/routing/git/commits{/sha}","comments_url":"https://api.github.com/repos/mmoadeli/routing/comments{/number}","issue_comment_url":"https://api.github.com/repos/mmoadeli/routing/issues/comments{/number}","contents_url":"https://api.github.com/repos/mmoadeli/routing/contents/{+path}","compare_url":"https://api.github.com/repos/mmoadeli/routing/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mmoadeli/routing/merges","archive_url":"https://api.github.com/repos/mmoadeli/routing/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mmoadeli/routing/downloads","issues_url":"https://api.github.com/repos/mmoadeli/routing/issues{/number}","pulls_url":"https://api.github.com/repos/mmoadeli/routing/pulls{/number}","milestones_url":"https://api.github.com/repos/mmoadeli/routing/milestones{/number}","notifications_url":"https://api.github.com/repos/mmoadeli/routing/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mmoadeli/routing/labels{/name}","releases_url":"https://api.github.com/repos/mmoadeli/routing/releases{/id}","created_at":"2015-04-24T14:05:09Z","updated_at":"2015-05-06T15:04:14Z","pushed_at":"2015-07-22T15:45:37Z","git_url":"git://github.com/mmoadeli/routing.git","ssh_url":"git@github.com:mmoadeli/routing.git","clone_url":"https://github.com/mmoadeli/routing.git","svn_url":"https://github.com/mmoadeli/routing","homepage":null,"size":3429,"stargazers_count":0,"watchers_count":0,"language":"Rust","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":"maidsafe:master","ref":"master","sha":"849943e849b761cd1b40a952d47c54a1b92426d0","user":{"login":"maidsafe","id":536423,"avatar_url":"https://avatars.githubusercontent.com/u/536423?v=3","gravatar_id":"","url":"https://api.github.com/users/maidsafe","html_url":"https://github.com/maidsafe","followers_url":"https://api.github.com/users/maidsafe/followers","following_url":"https://api.github.com/users/maidsafe/following{/other_user}","gists_url":"https://api.github.com/users/maidsafe/gists{/gist_id}","starred_url":"https://api.github.com/users/maidsafe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maidsafe/subscriptions","organizations_url":"https://api.github.com/users/maidsafe/orgs","repos_url":"https://api.github.com/users/maidsafe/repos","events_url":"https://api.github.com/users/maidsafe/events{/privacy}","received_events_url":"https://api.github.com/users/maidsafe/received_events","type":"Organization","site_admin":false},"repo":{"id":31270514,"name":"routing","full_name":"maidsafe/routing","owner":{"login":"maidsafe","id":536423,"avatar_url":"https://avatars.githubusercontent.com/u/536423?v=3","gravatar_id":"","url":"https://api.github.com/users/maidsafe","html_url":"https://github.com/maidsafe","followers_url":"https://api.github.com/users/maidsafe/followers","following_url":"https://api.github.com/users/maidsafe/following{/other_user}","gists_url":"https://api.github.com/users/maidsafe/gists{/gist_id}","starred_url":"https://api.github.com/users/maidsafe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maidsafe/subscriptions","organizations_url":"https://api.github.com/users/maidsafe/orgs","repos_url":"https://api.github.com/users/maidsafe/repos","events_url":"https://api.github.com/users/maidsafe/events{/privacy}","received_events_url":"https://api.github.com/users/maidsafe/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/maidsafe/routing","description":"Routing - specialised storage DHT","fork":false,"url":"https://api.github.com/repos/maidsafe/routing","forks_url":"https://api.github.com/repos/maidsafe/routing/forks","keys_url":"https://api.github.com/repos/maidsafe/routing/keys{/key_id}","collaborators_url":"https://api.github.com/repos/maidsafe/routing/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/maidsafe/routing/teams","hooks_url":"https://api.github.com/repos/maidsafe/routing/hooks","issue_events_url":"https://api.github.com/repos/maidsafe/routing/issues/events{/number}","events_url":"https://api.github.com/repos/maidsafe/routing/events","assignees_url":"https://api.github.com/repos/maidsafe/routing/assignees{/user}","branches_url":"https://api.github.com/repos/maidsafe/routing/branches{/branch}","tags_url":"https://api.github.com/repos/maidsafe/routing/tags","blobs_url":"https://api.github.com/repos/maidsafe/routing/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/maidsafe/routing/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/maidsafe/routing/git/refs{/sha}","trees_url":"https://api.github.com/repos/maidsafe/routing/git/trees{/sha}","statuses_url":"https://api.github.com/repos/maidsafe/routing/statuses/{sha}","languages_url":"https://api.github.com/repos/maidsafe/routing/languages","stargazers_url":"https://api.github.com/repos/maidsafe/routing/stargazers","contributors_url":"https://api.github.com/repos/maidsafe/routing/contributors","subscribers_url":"https://api.github.com/repos/maidsafe/routing/subscribers","subscription_url":"https://api.github.com/repos/maidsafe/routing/subscription","commits_url":"https://api.github.com/repos/maidsafe/routing/commits{/sha}","git_commits_url":"https://api.github.com/repos/maidsafe/routing/git/commits{/sha}","comments_url":"https://api.github.com/repos/maidsafe/routing/comments{/number}","issue_comment_url":"https://api.github.com/repos/maidsafe/routing/issues/comments{/number}","contents_url":"https://api.github.com/repos/maidsafe/routing/contents/{+path}","compare_url":"https://api.github.com/repos/maidsafe/routing/compare/{base}...{head}","merges_url":"https://api.github.com/repos/maidsafe/routing/merges","archive_url":"https://api.github.com/repos/maidsafe/routing/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/maidsafe/routing/downloads","issues_url":"https://api.github.com/repos/maidsafe/routing/issues{/number}","pulls_url":"https://api.github.com/repos/maidsafe/routing/pulls{/number}","milestones_url":"https://api.github.com/repos/maidsafe/routing/milestones{/number}","notifications_url":"https://api.github.com/repos/maidsafe/routing/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/maidsafe/routing/labels{/name}","releases_url":"https://api.github.com/repos/maidsafe/routing/releases{/id}","created_at":"2015-02-24T16:52:53Z","updated_at":"2015-07-22T19:34:06Z","pushed_at":"2015-07-22T15:45:42Z","git_url":"git://github.com/maidsafe/routing.git","ssh_url":"git@github.com:maidsafe/routing.git","clone_url":"https://github.com/maidsafe/routing.git","svn_url":"https://github.com/maidsafe/routing","homepage":null,"size":14899,"stargazers_count":39,"watchers_count":39,"language":"Rust","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":22,"mirror_url":null,"open_issues_count":25,"forks":22,"open_issues":25,"watchers":39,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/maidsafe/routing/pulls/499"},"html":{"href":"https://github.com/maidsafe/routing/pull/499"},"issue":{"href":"https://api.github.com/repos/maidsafe/routing/issues/499"},"comments":{"href":"https://api.github.com/repos/maidsafe/routing/issues/499/comments"},"review_comments":{"href":"https://api.github.com/repos/maidsafe/routing/pulls/499/comments"},"review_comment":{"href":"https://api.github.com/repos/maidsafe/routing/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/maidsafe/routing/pulls/499/commits"},"statuses":{"href":"https://api.github.com/repos/maidsafe/routing/statuses/16b2386f7176983c7edd7d767c3403a190e226ee"}}}}