{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/origin/issues/4743","labels_url":"https://api.github.com/repos/openshift/origin/issues/4743/labels{/name}","comments_url":"https://api.github.com/repos/openshift/origin/issues/4743/comments","events_url":"https://api.github.com/repos/openshift/origin/issues/4743/events","html_url":"https://github.com/openshift/origin/pull/4743","id":107546436,"number":4743,"title":"Follow up on #4660: new app error msg","user":{"login":"rhcarvalho","id":88819,"avatar_url":"https://avatars.githubusercontent.com/u/88819?v=3","gravatar_id":"","url":"https://api.github.com/users/rhcarvalho","html_url":"https://github.com/rhcarvalho","followers_url":"https://api.github.com/users/rhcarvalho/followers","following_url":"https://api.github.com/users/rhcarvalho/following{/other_user}","gists_url":"https://api.github.com/users/rhcarvalho/gists{/gist_id}","starred_url":"https://api.github.com/users/rhcarvalho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhcarvalho/subscriptions","organizations_url":"https://api.github.com/users/rhcarvalho/orgs","repos_url":"https://api.github.com/users/rhcarvalho/repos","events_url":"https://api.github.com/users/rhcarvalho/events{/privacy}","received_events_url":"https://api.github.com/users/rhcarvalho/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2015-09-21T16:10:51Z","updated_at":"2016-02-02T09:18:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openshift/origin/pulls/4743","html_url":"https://github.com/openshift/origin/pull/4743","diff_url":"https://github.com/openshift/origin/pull/4743.diff","patch_url":"https://github.com/openshift/origin/pull/4743.patch"},"body":"Given a JSON template with invalid syntax:\r\n\r\n``` \r\n$ cat bad-template.json\r\n{\r\n\"there should not be a trailing comma\": \"here\",\r\n}\r\n```\r\n\r\nBefore #4660, `oc new-app` produced this error message:\r\n\r\n```\r\n$ oc new-app bad-template.json\r\nerror: no image or template matched \"bad-template.json\"\r\n```\r\n\r\nAfter:\r\n\r\n```\r\n$ oc new-app bad-template.json\r\nerror: invalid character '}' looking for beginning of object key string\r\n```\r\n\r\nBut @smarterclayton [commented](https://github.com/openshift/origin/pull/4660#discussion_r39747207) that the implementation was leaking an abstraction, thus, this follow up."},"comment":{"url":"https://api.github.com/repos/openshift/origin/issues/comments/178468139","html_url":"https://github.com/openshift/origin/pull/4743#issuecomment-178468139","issue_url":"https://api.github.com/repos/openshift/origin/issues/4743","id":178468139,"user":{"login":"rhcarvalho","id":88819,"avatar_url":"https://avatars.githubusercontent.com/u/88819?v=3","gravatar_id":"","url":"https://api.github.com/users/rhcarvalho","html_url":"https://github.com/rhcarvalho","followers_url":"https://api.github.com/users/rhcarvalho/followers","following_url":"https://api.github.com/users/rhcarvalho/following{/other_user}","gists_url":"https://api.github.com/users/rhcarvalho/gists{/gist_id}","starred_url":"https://api.github.com/users/rhcarvalho/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhcarvalho/subscriptions","organizations_url":"https://api.github.com/users/rhcarvalho/orgs","repos_url":"https://api.github.com/users/rhcarvalho/repos","events_url":"https://api.github.com/users/rhcarvalho/events{/privacy}","received_events_url":"https://api.github.com/users/rhcarvalho/received_events","type":"User","site_admin":false},"created_at":"2016-02-02T09:18:22Z","updated_at":"2016-02-02T09:18:22Z","body":"(note to self) this PR might simply die after #6670 merges."}}