search

Financial-Times / tako

🐙 A GitHub App that provides an API listing the repositories it is installed on.
MIT License
4 stars 0 forks source link

The time when Tako had no repositories #114

Closed sjparkinson closed 5 years ago

sjparkinson commented 5 years ago

Looks like Tako got into a state where it was trying to use an expired auth token to make requests to GitHub.

It should be refreshed by Probot before it expires, however if some of our logic takes too long to run with a given context in theory we could make a call to GitHub with an expired token.

Related error in Sentry...

HttpError: Bad credentials
  File "/app/node_modules/@octokit/request/lib/request.js", line 55, col 27, in response.text.then.message
    const error = new HttpError(message, status, headers, requestOptions)
  File "internal/process/next_tick.js", line 68, col 7, in process._tickCallback

https://sentry.io/organizations/nextftcom/issues/906555985/

Potentially relates to https://github.com/probot/probot/issues/637.

sjparkinson commented 5 years ago

Initial report by @adambraimbridge in Slack, https://financialtimes.slack.com/archives/CD689QU2K/p1551092355000900.

We "resolved" the issue by manually restarting the Heroku dyno from the app's dashboard.

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.