{"action":"opened","issue":{"url":"https://api.github.com/repos/KAIST-IS521/TeamThree/issues/32","repository_url":"https://api.github.com/repos/KAIST-IS521/TeamThree","labels_url":"https://api.github.com/repos/KAIST-IS521/TeamThree/issues/32/labels{/name}","comments_url":"https://api.github.com/repos/KAIST-IS521/TeamThree/issues/32/comments","events_url":"https://api.github.com/repos/KAIST-IS521/TeamThree/issues/32/events","html_url":"https://github.com/KAIST-IS521/TeamThree/issues/32","id":230437652,"number":32,"title":"handshake() implemented wrong","user":{"login":"ian0371","id":7467216,"avatar_url":"https://avatars2.githubusercontent.com/u/7467216?v=3","gravatar_id":"","url":"https://api.github.com/users/ian0371","html_url":"https://github.com/ian0371","followers_url":"https://api.github.com/users/ian0371/followers","following_url":"https://api.github.com/users/ian0371/following{/other_user}","gists_url":"https://api.github.com/users/ian0371/gists{/gist_id}","starred_url":"https://api.github.com/users/ian0371/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ian0371/subscriptions","organizations_url":"https://api.github.com/users/ian0371/orgs","repos_url":"https://api.github.com/users/ian0371/repos","events_url":"https://api.github.com/users/ian0371/events{/privacy}","received_events_url":"https://api.github.com/users/ian0371/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-05-22T15:42:30Z","updated_at":"2017-05-22T15:42:30Z","closed_at":null,"body":"Based on the fact that handshake() generates random number and that `send` is followed by `recv` as below:\r\n```\r\naio_send(sock, buffer, 4096);\r\n...\r\nnbytes = aio_recv(sock,buffer, 4096);\r\n```\r\nI think handshake() is implemented from server's viewpoint, when it must be used by clients trying to be authenticated.\r\nAlso, since the function is incorrectly implemented and due to lack of protocol that is still not specified in issue #23, other teams could not get started on authentication part."}}
{"actor":{"display_login":"ian0371"}}