{"action":"created","issue":{"url":"https://api.github.com/repos/IntersectAustralia/hcsvlab/issues/12","labels_url":"https://api.github.com/repos/IntersectAustralia/hcsvlab/issues/12/labels{/name}","comments_url":"https://api.github.com/repos/IntersectAustralia/hcsvlab/issues/12/comments","events_url":"https://api.github.com/repos/IntersectAustralia/hcsvlab/issues/12/events","html_url":"https://github.com/IntersectAustralia/hcsvlab/pull/12","id":104595899,"number":12,"title":"Hcsvlab 1019","user":{"login":"shaun-intersect","id":10394411,"avatar_url":"https://avatars.githubusercontent.com/u/10394411?v=3","gravatar_id":"","url":"https://api.github.com/users/shaun-intersect","html_url":"https://github.com/shaun-intersect","followers_url":"https://api.github.com/users/shaun-intersect/followers","following_url":"https://api.github.com/users/shaun-intersect/following{/other_user}","gists_url":"https://api.github.com/users/shaun-intersect/gists{/gist_id}","starred_url":"https://api.github.com/users/shaun-intersect/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shaun-intersect/subscriptions","organizations_url":"https://api.github.com/users/shaun-intersect/orgs","repos_url":"https://api.github.com/users/shaun-intersect/repos","events_url":"https://api.github.com/users/shaun-intersect/events{/privacy}","received_events_url":"https://api.github.com/users/shaun-intersect/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"dyazbek-intersect","id":1744043,"avatar_url":"https://avatars.githubusercontent.com/u/1744043?v=3","gravatar_id":"","url":"https://api.github.com/users/dyazbek-intersect","html_url":"https://github.com/dyazbek-intersect","followers_url":"https://api.github.com/users/dyazbek-intersect/followers","following_url":"https://api.github.com/users/dyazbek-intersect/following{/other_user}","gists_url":"https://api.github.com/users/dyazbek-intersect/gists{/gist_id}","starred_url":"https://api.github.com/users/dyazbek-intersect/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dyazbek-intersect/subscriptions","organizations_url":"https://api.github.com/users/dyazbek-intersect/orgs","repos_url":"https://api.github.com/users/dyazbek-intersect/repos","events_url":"https://api.github.com/users/dyazbek-intersect/events{/privacy}","received_events_url":"https://api.github.com/users/dyazbek-intersect/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-09-03T00:12:53Z","updated_at":"2015-09-04T08:24:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/IntersectAustralia/hcsvlab/pulls/12","html_url":"https://github.com/IntersectAustralia/hcsvlab/pull/12","diff_url":"https://github.com/IntersectAustralia/hcsvlab/pull/12.diff","patch_url":"https://github.com/IntersectAustralia/hcsvlab/pull/12.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/IntersectAustralia/hcsvlab/issues/comments/137676795","html_url":"https://github.com/IntersectAustralia/hcsvlab/pull/12#issuecomment-137676795","issue_url":"https://api.github.com/repos/IntersectAustralia/hcsvlab/issues/12","id":137676795,"user":{"login":"shaun-intersect","id":10394411,"avatar_url":"https://avatars.githubusercontent.com/u/10394411?v=3","gravatar_id":"","url":"https://api.github.com/users/shaun-intersect","html_url":"https://github.com/shaun-intersect","followers_url":"https://api.github.com/users/shaun-intersect/followers","following_url":"https://api.github.com/users/shaun-intersect/following{/other_user}","gists_url":"https://api.github.com/users/shaun-intersect/gists{/gist_id}","starred_url":"https://api.github.com/users/shaun-intersect/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shaun-intersect/subscriptions","organizations_url":"https://api.github.com/users/shaun-intersect/orgs","repos_url":"https://api.github.com/users/shaun-intersect/repos","events_url":"https://api.github.com/users/shaun-intersect/events{/privacy}","received_events_url":"https://api.github.com/users/shaun-intersect/received_events","type":"User","site_admin":false},"created_at":"2015-09-04T08:24:06Z","updated_at":"2015-09-04T08:24:06Z","body":"Suggestions given by Dan to:\r\n- validate and catch case when malformed JSON is given in the request, it appears that this is currently unhandled.\r\n- perform all validations upfront before doing any program logic, including the validations that uploaded files don't already exist on the server.\r\n- provide some warning message if files already exist on the server (if it is meant to skip existing files rather than exit/fail when one is encountered)."}}