Abuse rate limits

[bkeepers]

And so it begins…

https://sentry.io/github-integrations/slack/issues/468663816/

HttpError: {
  "documentation_url": "https://developer.github.com/v3/#abuse-rate-limits",
  "message": "You have triggered an abuse detection mechanism. Please wait a few minutes before you try again."
}

  File "/app/node_modules/@octokit/rest/lib/request/request.js", line 73, in IncomingMessage.response.on
    reject(new HttpError(data, response.statusCode, response.headers))
  File "events.js", line 111, in emitNone
  File "events.js", line 208, in IncomingMessage.emit
  File "_stream_readable.js", line 1055, in endReadableNT
  File "internal/process/next_tick.js", line 138, in _combinedTickCallback

Unfortunately the exception is not super helpful about what is trigger this error. We're going to be seeing a lot of these over time, so it'll be important that we have a good strategy for handling them:

• [ ] Get full stack traces in sentry for these errors
• [ ] Instrument the GitHub client so we can see where we're making excessive API calls
• [ ] Gracefully handle these errors and give users insight into why they are failing and how they can avoid them

[ajendrex]

Hi guys,

I'm using this integration to have PRs creation notified in a slack channel, but some PRs are never notified; can that be related to this issue?

[wilhelmklopp]

@ajendrex the reason for that was likely replication lag which was addressed in #494