{"action":"created","issue":{"url":"https://api.github.com/repos/serilog-web/classic/issues/24","labels_url":"https://api.github.com/repos/serilog-web/classic/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/serilog-web/classic/issues/24/comments","events_url":"https://api.github.com/repos/serilog-web/classic/issues/24/events","html_url":"https://github.com/serilog-web/classic/pull/24","id":125806465,"number":24,"title":"Log webapi exceptions","user":{"login":"teyc","id":2076,"avatar_url":"https://avatars.githubusercontent.com/u/2076?v=3","gravatar_id":"","url":"https://api.github.com/users/teyc","html_url":"https://github.com/teyc","followers_url":"https://api.github.com/users/teyc/followers","following_url":"https://api.github.com/users/teyc/following{/other_user}","gists_url":"https://api.github.com/users/teyc/gists{/gist_id}","starred_url":"https://api.github.com/users/teyc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/teyc/subscriptions","organizations_url":"https://api.github.com/users/teyc/orgs","repos_url":"https://api.github.com/users/teyc/repos","events_url":"https://api.github.com/users/teyc/events{/privacy}","received_events_url":"https://api.github.com/users/teyc/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-01-10T07:54:00Z","updated_at":"2016-01-12T22:34:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/serilog-web/classic/pulls/24","html_url":"https://github.com/serilog-web/classic/pull/24","diff_url":"https://github.com/serilog-web/classic/pull/24.diff","patch_url":"https://github.com/serilog-web/classic/pull/24.patch"},"body":"Hi Nick,\r\n\r\nI've got a fix for the issue where WebApi exceptions were not getting logged. However, to do it within the SerilogWeb.Classic package requires an additional dependency on `WebApi.Core`, which breaks the principle for pay only when you need it. \r\n\r\nThe alternatives are\r\n-  a single file which the user adds to their project, \r\n- or a bit of documentation in the wiki\r\n- or a separate NuGet package.\r\n\r\nWhat do you think?"},"comment":{"url":"https://api.github.com/repos/serilog-web/classic/issues/comments/171082022","html_url":"https://github.com/serilog-web/classic/pull/24#issuecomment-171082022","issue_url":"https://api.github.com/repos/serilog-web/classic/issues/24","id":171082022,"user":{"login":"teyc","id":2076,"avatar_url":"https://avatars.githubusercontent.com/u/2076?v=3","gravatar_id":"","url":"https://api.github.com/users/teyc","html_url":"https://github.com/teyc","followers_url":"https://api.github.com/users/teyc/followers","following_url":"https://api.github.com/users/teyc/following{/other_user}","gists_url":"https://api.github.com/users/teyc/gists{/gist_id}","starred_url":"https://api.github.com/users/teyc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/teyc/subscriptions","organizations_url":"https://api.github.com/users/teyc/orgs","repos_url":"https://api.github.com/users/teyc/repos","events_url":"https://api.github.com/users/teyc/events{/privacy}","received_events_url":"https://api.github.com/users/teyc/received_events","type":"User","site_admin":false},"created_at":"2016-01-12T22:34:30Z","updated_at":"2016-01-12T22:34:30Z","body":"Choosing names can be so hard. :)\n\nI defer to you. *SerilogWeb.Clasic.WebApi* it shall be, as it sounds more\nin line with NuGet naming practices.\n\nOn Wed, Jan 13, 2016 at 8:10 AM, Nicholas Blumhardt <\nnotifications@github.com> wrote:\n\n> BTW, I'm thinking *SerilogWeb.Classic.WebApi* would be a reasonable\n> package name - rather than \"and\" it's more of a \"subcomponent\" in my mind.\n> Open to ideas though!\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/serilog-web/classic/pull/24#issuecomment-171076134>.\n>\n"}}