{"action":"created","issue":{"url":"https://api.github.com/repos/ClusterHQ/eliot/issues/237","labels_url":"https://api.github.com/repos/ClusterHQ/eliot/issues/237/labels{/name}","comments_url":"https://api.github.com/repos/ClusterHQ/eliot/issues/237/comments","events_url":"https://api.github.com/repos/ClusterHQ/eliot/issues/237/events","html_url":"https://github.com/ClusterHQ/eliot/pull/237","id":118658401,"number":237,"title":"Extract fields from errors","user":{"login":"itamarst","id":3266662,"avatar_url":"https://avatars.githubusercontent.com/u/3266662?v=3","gravatar_id":"","url":"https://api.github.com/users/itamarst","html_url":"https://github.com/itamarst","followers_url":"https://api.github.com/users/itamarst/followers","following_url":"https://api.github.com/users/itamarst/following{/other_user}","gists_url":"https://api.github.com/users/itamarst/gists{/gist_id}","starred_url":"https://api.github.com/users/itamarst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/itamarst/subscriptions","organizations_url":"https://api.github.com/users/itamarst/orgs","repos_url":"https://api.github.com/users/itamarst/repos","events_url":"https://api.github.com/users/itamarst/events{/privacy}","received_events_url":"https://api.github.com/users/itamarst/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-11-24T17:09:06Z","updated_at":"2015-11-25T14:46:03Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ClusterHQ/eliot/pulls/237","html_url":"https://github.com/ClusterHQ/eliot/pull/237","diff_url":"https://github.com/ClusterHQ/eliot/pull/237.diff","patch_url":"https://github.com/ClusterHQ/eliot/pull/237.patch"},"body":"Fixes #236.\r\n\r\nThis will allow e.g. including HTTP response code as a field in failed actions due to `flocker.restapi._error.BadRequest`."},"comment":{"url":"https://api.github.com/repos/ClusterHQ/eliot/issues/comments/159629082","html_url":"https://github.com/ClusterHQ/eliot/pull/237#issuecomment-159629082","issue_url":"https://api.github.com/repos/ClusterHQ/eliot/issues/237","id":159629082,"user":{"login":"itamarst","id":3266662,"avatar_url":"https://avatars.githubusercontent.com/u/3266662?v=3","gravatar_id":"","url":"https://api.github.com/users/itamarst","html_url":"https://github.com/itamarst","followers_url":"https://api.github.com/users/itamarst/followers","following_url":"https://api.github.com/users/itamarst/following{/other_user}","gists_url":"https://api.github.com/users/itamarst/gists{/gist_id}","starred_url":"https://api.github.com/users/itamarst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/itamarst/subscriptions","organizations_url":"https://api.github.com/users/itamarst/orgs","repos_url":"https://api.github.com/users/itamarst/repos","events_url":"https://api.github.com/users/itamarst/events{/privacy}","received_events_url":"https://api.github.com/users/itamarst/received_events","type":"User","site_admin":false},"created_at":"2015-11-25T14:46:03Z","updated_at":"2015-11-25T14:46:03Z","body":"The type system is a tricky one, yeah... Requiring every ActionType to specify every possible exception-extracted field seems unreasonable, seems better to do it on per-exception level.\r\n\r\nHowever, there's still issue with adding these new fields to `eliot:traceback` since it has a MessageType with fixed fields...\r\n"}}