{"action":"created","issue":{"url":"https://api.github.com/repos/interagent/pliny/issues/264","repository_url":"https://api.github.com/repos/interagent/pliny","labels_url":"https://api.github.com/repos/interagent/pliny/issues/264/labels{/name}","comments_url":"https://api.github.com/repos/interagent/pliny/issues/264/comments","events_url":"https://api.github.com/repos/interagent/pliny/issues/264/events","html_url":"https://github.com/interagent/pliny/pull/264","id":150003849,"number":264,"title":"Disable Rollbar monkey patching","user":{"login":"gudmundur","id":35417,"avatar_url":"https://avatars.githubusercontent.com/u/35417?v=3","gravatar_id":"","url":"https://api.github.com/users/gudmundur","html_url":"https://github.com/gudmundur","followers_url":"https://api.github.com/users/gudmundur/followers","following_url":"https://api.github.com/users/gudmundur/following{/other_user}","gists_url":"https://api.github.com/users/gudmundur/gists{/gist_id}","starred_url":"https://api.github.com/users/gudmundur/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gudmundur/subscriptions","organizations_url":"https://api.github.com/users/gudmundur/orgs","repos_url":"https://api.github.com/users/gudmundur/repos","events_url":"https://api.github.com/users/gudmundur/events{/privacy}","received_events_url":"https://api.github.com/users/gudmundur/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-04-21T08:31:28Z","updated_at":"2016-04-26T09:23:23Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/interagent/pliny/pulls/264","html_url":"https://github.com/interagent/pliny/pull/264","diff_url":"https://github.com/interagent/pliny/pull/264.diff","patch_url":"https://github.com/interagent/pliny/pull/264.patch"},"body":"During investigation of an incident I noticed that Pliny has been double posting to Rollbar for a while. I traced this to a missing `disable_monkey_patch` as a part of #110 a while back.\r\n\r\n**Note** Bringing this change in might have the side effect of error reporting being disabled in certain places, for example in Sidekiq workers and other `bin/` scripts."},"comment":{"url":"https://api.github.com/repos/interagent/pliny/issues/comments/214681393","html_url":"https://github.com/interagent/pliny/pull/264#issuecomment-214681393","issue_url":"https://api.github.com/repos/interagent/pliny/issues/264","id":214681393,"user":{"login":"gudmundur","id":35417,"avatar_url":"https://avatars.githubusercontent.com/u/35417?v=3","gravatar_id":"","url":"https://api.github.com/users/gudmundur","html_url":"https://github.com/gudmundur","followers_url":"https://api.github.com/users/gudmundur/followers","following_url":"https://api.github.com/users/gudmundur/following{/other_user}","gists_url":"https://api.github.com/users/gudmundur/gists{/gist_id}","starred_url":"https://api.github.com/users/gudmundur/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gudmundur/subscriptions","organizations_url":"https://api.github.com/users/gudmundur/orgs","repos_url":"https://api.github.com/users/gudmundur/repos","events_url":"https://api.github.com/users/gudmundur/events{/privacy}","received_events_url":"https://api.github.com/users/gudmundur/received_events","type":"User","site_admin":false},"created_at":"2016-04-26T09:23:23Z","updated_at":"2016-04-26T09:23:23Z","body":"@brandur @pedro To refresh my memory before bringing this in: What is the motivation for disabling monkey patching? If Pliny is biased towards Rollbar, wouldn't we want it enabled by default everywhere? Even though it means monkey patching? If middleware ordering is all we care about, there's now the option of just disabling monkey patching for rack, `disable_rack_monkey_patch`."}}