{"action":"created","issue":{"url":"https://api.github.com/repos/cloudfoundry-incubator/diego-ssh/issues/12","labels_url":"https://api.github.com/repos/cloudfoundry-incubator/diego-ssh/issues/12/labels{/name}","comments_url":"https://api.github.com/repos/cloudfoundry-incubator/diego-ssh/issues/12/comments","events_url":"https://api.github.com/repos/cloudfoundry-incubator/diego-ssh/issues/12/events","html_url":"https://github.com/cloudfoundry-incubator/diego-ssh/pull/12","id":119993939,"number":12,"title":"Simplify SHA1/MD5 fingerprint generators","user":{"login":"krishicks","id":79357,"avatar_url":"https://avatars.githubusercontent.com/u/79357?v=3","gravatar_id":"","url":"https://api.github.com/users/krishicks","html_url":"https://github.com/krishicks","followers_url":"https://api.github.com/users/krishicks/followers","following_url":"https://api.github.com/users/krishicks/following{/other_user}","gists_url":"https://api.github.com/users/krishicks/gists{/gist_id}","starred_url":"https://api.github.com/users/krishicks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/krishicks/subscriptions","organizations_url":"https://api.github.com/users/krishicks/orgs","repos_url":"https://api.github.com/users/krishicks/repos","events_url":"https://api.github.com/users/krishicks/events{/privacy}","received_events_url":"https://api.github.com/users/krishicks/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-12-02T17:27:14Z","updated_at":"2015-12-02T20:48:15Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cloudfoundry-incubator/diego-ssh/pulls/12","html_url":"https://github.com/cloudfoundry-incubator/diego-ssh/pull/12","diff_url":"https://github.com/cloudfoundry-incubator/diego-ssh/pull/12.diff","patch_url":"https://github.com/cloudfoundry-incubator/diego-ssh/pull/12.patch"},"body":"- Remove the very curious usage of '[:]', which was used because the hex\r\n  function requires []byte, but md5/sha1.Sum give you [Size]byte, and\r\n  [:] will return []byte.\r\n- Stop iterating through the bytes to construct the colonized string;\r\n  use fmt.Sprintf(\"% x\", data) as shown in Golang docs (and replace the\r\n  spaces with colons): https://play.golang.org/p/e7W5FVB-hW"},"comment":{"url":"https://api.github.com/repos/cloudfoundry-incubator/diego-ssh/issues/comments/161429674","html_url":"https://github.com/cloudfoundry-incubator/diego-ssh/pull/12#issuecomment-161429674","issue_url":"https://api.github.com/repos/cloudfoundry-incubator/diego-ssh/issues/12","id":161429674,"user":{"login":"ematpl","id":5836757,"avatar_url":"https://avatars.githubusercontent.com/u/5836757?v=3","gravatar_id":"","url":"https://api.github.com/users/ematpl","html_url":"https://github.com/ematpl","followers_url":"https://api.github.com/users/ematpl/followers","following_url":"https://api.github.com/users/ematpl/following{/other_user}","gists_url":"https://api.github.com/users/ematpl/gists{/gist_id}","starred_url":"https://api.github.com/users/ematpl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ematpl/subscriptions","organizations_url":"https://api.github.com/users/ematpl/orgs","repos_url":"https://api.github.com/users/ematpl/repos","events_url":"https://api.github.com/users/ematpl/events{/privacy}","received_events_url":"https://api.github.com/users/ematpl/received_events","type":"User","site_admin":false},"created_at":"2015-12-02T20:48:14Z","updated_at":"2015-12-02T20:48:14Z","body":"Great, thanks, @krishicks! Looks much simpler! Does the `strings.Replace`/`fmt.Sprintf` combo deserve extraction into its own private helper function?\r\n\r\nBest,\r\nEric"}}