{"action":"created","issue":{"url":"https://api.github.com/repos/guardian/grid/issues/1392","labels_url":"https://api.github.com/repos/guardian/grid/issues/1392/labels{/name}","comments_url":"https://api.github.com/repos/guardian/grid/issues/1392/comments","events_url":"https://api.github.com/repos/guardian/grid/issues/1392/events","html_url":"https://github.com/guardian/grid/pull/1392","id":112835980,"number":1392,"title":"Better Lambda fail states and reporting","user":{"login":"kenoir","id":953792,"avatar_url":"https://avatars.githubusercontent.com/u/953792?v=3","gravatar_id":"","url":"https://api.github.com/users/kenoir","html_url":"https://github.com/kenoir","followers_url":"https://api.github.com/users/kenoir/followers","following_url":"https://api.github.com/users/kenoir/following{/other_user}","gists_url":"https://api.github.com/users/kenoir/gists{/gist_id}","starred_url":"https://api.github.com/users/kenoir/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kenoir/subscriptions","organizations_url":"https://api.github.com/users/kenoir/orgs","repos_url":"https://api.github.com/users/kenoir/repos","events_url":"https://api.github.com/users/kenoir/events{/privacy}","received_events_url":"https://api.github.com/users/kenoir/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-22T16:02:27Z","updated_at":"2015-10-26T10:15:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/guardian/grid/pulls/1392","html_url":"https://github.com/guardian/grid/pull/1392","diff_url":"https://github.com/guardian/grid/pull/1392.diff","patch_url":"https://github.com/guardian/grid/pull/1392.patch"},"body":"- Don't fail Lambda function when upload fails to avoid Lambda retries\r\n- Report successful uploads & failures to CW"},"comment":{"url":"https://api.github.com/repos/guardian/grid/issues/comments/151090562","html_url":"https://github.com/guardian/grid/pull/1392#issuecomment-151090562","issue_url":"https://api.github.com/repos/guardian/grid/issues/1392","id":151090562,"user":{"login":"kenoir","id":953792,"avatar_url":"https://avatars.githubusercontent.com/u/953792?v=3","gravatar_id":"","url":"https://api.github.com/users/kenoir","html_url":"https://github.com/kenoir","followers_url":"https://api.github.com/users/kenoir/followers","following_url":"https://api.github.com/users/kenoir/following{/other_user}","gists_url":"https://api.github.com/users/kenoir/gists{/gist_id}","starred_url":"https://api.github.com/users/kenoir/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kenoir/subscriptions","organizations_url":"https://api.github.com/users/kenoir/orgs","repos_url":"https://api.github.com/users/kenoir/repos","events_url":"https://api.github.com/users/kenoir/events{/privacy}","received_events_url":"https://api.github.com/users/kenoir/received_events","type":"User","site_admin":false},"created_at":"2015-10-26T10:15:49Z","updated_at":"2015-10-26T10:15:49Z","body":"@seb addressed what happens in error states above, (it's as we discussed I think). I decided not to fail fast on non jpeg mime types as it's not straightforward to add and can be addressed by modifying the Lambda trigger events if necessary (it's possible to only send events on *.jpg/*.jpeg files). Though our aspirations to handle pngs and the low frequency of \"bad\" files make me feel that it's unnecessary complexity."}}