{"action":"created","issue":{"url":"https://api.github.com/repos/guardian/frontend/issues/8322","labels_url":"https://api.github.com/repos/guardian/frontend/issues/8322/labels{/name}","comments_url":"https://api.github.com/repos/guardian/frontend/issues/8322/comments","events_url":"https://api.github.com/repos/guardian/frontend/issues/8322/events","html_url":"https://github.com/guardian/frontend/pull/8322","id":57593259,"number":8322,"title":"refactor of missing video encodings job","user":{"login":"NathanielBennett","id":986155,"avatar_url":"https://avatars.githubusercontent.com/u/986155?v=3","gravatar_id":"","url":"https://api.github.com/users/NathanielBennett","html_url":"https://github.com/NathanielBennett","followers_url":"https://api.github.com/users/NathanielBennett/followers","following_url":"https://api.github.com/users/NathanielBennett/following{/other_user}","gists_url":"https://api.github.com/users/NathanielBennett/gists{/gist_id}","starred_url":"https://api.github.com/users/NathanielBennett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NathanielBennett/subscriptions","organizations_url":"https://api.github.com/users/NathanielBennett/orgs","repos_url":"https://api.github.com/users/NathanielBennett/repos","events_url":"https://api.github.com/users/NathanielBennett/events{/privacy}","received_events_url":"https://api.github.com/users/NathanielBennett/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-02-13T12:28:03Z","updated_at":"2015-02-18T12:13:28Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/guardian/frontend/pulls/8322","html_url":"https://github.com/guardian/frontend/pull/8322","diff_url":"https://github.com/guardian/frontend/pull/8322.diff","patch_url":"https://github.com/guardian/frontend/pull/8322.patch"},"body":"This is a refactor of this: https://github.com/guardian/frontend/pull/8154 which does three things: \r\n\r\n1) Uses a slightly different api query: http://content.guardianapis.com/search?api-key=test&show-assets=all&tag=type/video&page-size=100 and uses the assets parse sub-document to locate the encodings for each video encoding. I've updated some variable names and tidied up the indentation because the scala for this is a little bit perverted ... \r\n\r\n2) Strips out any newline characters in the individual encoding urls: Some of the octopus urls have newline characters embedded in them which (a) causes the agent to throw error and (b) is a possible cause for video assents being unavailable in the first place\r\n\r\n3) Hardcodes the name of of the SNS topic which missing encoding notifications get posted to. This is because previously, the topic created by cloudformation gets recreated each time someone cloudforms.   "},"comment":{"url":"https://api.github.com/repos/guardian/frontend/issues/comments/74854657","html_url":"https://github.com/guardian/frontend/pull/8322#issuecomment-74854657","issue_url":"https://api.github.com/repos/guardian/frontend/issues/8322","id":74854657,"user":{"login":"gklopper","id":76709,"avatar_url":"https://avatars.githubusercontent.com/u/76709?v=3","gravatar_id":"","url":"https://api.github.com/users/gklopper","html_url":"https://github.com/gklopper","followers_url":"https://api.github.com/users/gklopper/followers","following_url":"https://api.github.com/users/gklopper/following{/other_user}","gists_url":"https://api.github.com/users/gklopper/gists{/gist_id}","starred_url":"https://api.github.com/users/gklopper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gklopper/subscriptions","organizations_url":"https://api.github.com/users/gklopper/orgs","repos_url":"https://api.github.com/users/gklopper/repos","events_url":"https://api.github.com/users/gklopper/events{/privacy}","received_events_url":"https://api.github.com/users/gklopper/received_events","type":"User","site_admin":false},"created_at":"2015-02-18T12:13:28Z","updated_at":"2015-02-18T12:13:28Z","body":":+1: "}}