hubot-slack icon indicating copy to clipboard operation
hubot-slack copied to clipboard

Request: allow detecting when hubot-slack has disconnected and will not reconnect

Open mistydemeo opened this issue 7 years ago • 7 comments

Description

While hubot-slack has support for reconnecting when the connection to Slack is lost, there is a limit to the number of reconnection attempts. If several reconnection attempts fail, the bot will simply hang without a connection. This causes trouble for our process manager, since the client staying open means we don't end up automatically restart it. When this has happened for us a few times lately, we had to manually notice and restart the service in order to get it back up.

I'd like to see some form of event be emitted in this condition to allow us to determine when this has happened so that we can terminate the process and let the process manager restart it.

What type of issue is this? (place an x in one of the [ ])

  • [ ] bug
  • [x] enhancement (feature request)
  • [ ] question
  • [ ] documentation related
  • [ ] testing related
  • [ ] discussion

Requirements (place an x in each of the [ ])

  • [x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
  • [x] I've read and agree to the Code of Conduct.
  • [x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report

Filling out the following details about bugs will help us solve your issue sooner.

Reproducible in:

hubot-slack version: 4.5.5

node version: v8.2.0

OS version(s): Debian jessie

mistydemeo avatar Oct 04 '18 19:10 mistydemeo

related: https://github.com/slackapi/hubot-slack/pull/215

it looks like at some point in this package's history, there was an environment variable used to configure whether the process would terminate in this condition, not just fire an event. should we reintroduce that configuration? fire the event? both?

aoberoi avatar Oct 08 '18 01:10 aoberoi

It would be really nice to get this back into Hubot. We have a major issue at the moment with our hubot just disconnecting randomly from our slack, and there is no way to detect this so that we can restart the docker container it's running on.

ben-nat-wallis avatar Oct 23 '18 07:10 ben-nat-wallis

@ben-nat-wallis

For our hubot, I workaround at the moment with a cron script that checks the logs for the reconnect message. Then kill/restart the container 🤷‍♂

CHECK_STATUS=`docker logs $CONTAINER_ID | grep "INFO Slack client closed, waiting for reconnect" | wc -l

charliekump-wf avatar May 03 '19 15:05 charliekump-wf

👋 It looks like this issue has been open for 30 days with no activity. We'll mark this as stale for now, and wait 10 days for an update or for further comment before closing this issue out.

github-actions[bot] avatar Dec 05 '21 00:12 github-actions[bot]

There's been no activity because it was never fixed. I opened this in 2018.

mistydemeo avatar Dec 05 '21 01:12 mistydemeo

Hi @mistydemeo, sorry for the false alert. It was not an intentional behavior of the triage bot that we recently introduced recently. I've marked this issue as "auto-triage-skip".

seratch avatar Dec 05 '21 02:12 seratch

Hey guys any update on this issue?

Im having the same problem.

Suddenly hubot loses connectivity to slack and nothing shows on the logs.

caiocrivellente avatar Mar 03 '22 17:03 caiocrivellente