{"action":"created","issue":{"url":"https://api.github.com/repos/rlmcpherson/s3gof3r/issues/50","labels_url":"https://api.github.com/repos/rlmcpherson/s3gof3r/issues/50/labels{/name}","comments_url":"https://api.github.com/repos/rlmcpherson/s3gof3r/issues/50/comments","events_url":"https://api.github.com/repos/rlmcpherson/s3gof3r/issues/50/events","html_url":"https://github.com/rlmcpherson/s3gof3r/pull/50","id":55232081,"number":50,"title":"Handle cases where etag is missing in gracefully.","user":{"login":"subramanyamchitti","id":6935152,"avatar_url":"https://avatars.githubusercontent.com/u/6935152?v=3","gravatar_id":"","url":"https://api.github.com/users/subramanyamchitti","html_url":"https://github.com/subramanyamchitti","followers_url":"https://api.github.com/users/subramanyamchitti/followers","following_url":"https://api.github.com/users/subramanyamchitti/following{/other_user}","gists_url":"https://api.github.com/users/subramanyamchitti/gists{/gist_id}","starred_url":"https://api.github.com/users/subramanyamchitti/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/subramanyamchitti/subscriptions","organizations_url":"https://api.github.com/users/subramanyamchitti/orgs","repos_url":"https://api.github.com/users/subramanyamchitti/repos","events_url":"https://api.github.com/users/subramanyamchitti/events{/privacy}","received_events_url":"https://api.github.com/users/subramanyamchitti/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-23T01:04:15Z","updated_at":"2015-01-23T04:26:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/rlmcpherson/s3gof3r/pulls/50","html_url":"https://github.com/rlmcpherson/s3gof3r/pull/50","diff_url":"https://github.com/rlmcpherson/s3gof3r/pull/50.diff","patch_url":"https://github.com/rlmcpherson/s3gof3r/pull/50.patch"},"body":"The code as written doesn't handle cases where the etag is missing gracefully. This commit checks for the length of etag before indexing into it, to prevent index out of bounds panics.\r\n"},"comment":{"url":"https://api.github.com/repos/rlmcpherson/s3gof3r/issues/comments/71146004","html_url":"https://github.com/rlmcpherson/s3gof3r/pull/50#issuecomment-71146004","issue_url":"https://api.github.com/repos/rlmcpherson/s3gof3r/issues/50","id":71146004,"user":{"login":"rlmcpherson","id":1706273,"avatar_url":"https://avatars.githubusercontent.com/u/1706273?v=3","gravatar_id":"","url":"https://api.github.com/users/rlmcpherson","html_url":"https://github.com/rlmcpherson","followers_url":"https://api.github.com/users/rlmcpherson/followers","following_url":"https://api.github.com/users/rlmcpherson/following{/other_user}","gists_url":"https://api.github.com/users/rlmcpherson/gists{/gist_id}","starred_url":"https://api.github.com/users/rlmcpherson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rlmcpherson/subscriptions","organizations_url":"https://api.github.com/users/rlmcpherson/orgs","repos_url":"https://api.github.com/users/rlmcpherson/repos","events_url":"https://api.github.com/users/rlmcpherson/events{/privacy}","received_events_url":"https://api.github.com/users/rlmcpherson/received_events","type":"User","site_admin":false},"created_at":"2015-01-23T04:26:35Z","updated_at":"2015-01-23T04:26:35Z","body":"Do you know why the etag would be missing? It's documented as included in the response and is required to be sent on CompleteMultiPartUpload: http://docs.aws.amazon.com/AmazonS3/latest/API/mpUploadComplete.html"}}