Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No longer exit on client close #177

Merged
merged 1 commit into from
Apr 15, 2015

Conversation

davefp
Copy link
Contributor

@davefp davefp commented Apr 10, 2015

Fixes #127

@davefp
Copy link
Contributor Author

davefp commented Apr 10, 2015

I've been unable to 'really' test this as simulating network problems by simply severing my local wifi connection results in an error on reconnect which also causes the process to exit.

Suggestions on better tools/strategies to test this are welcome! In the meantime I'm going to be running my branch on a production hubot to see if any problems arise.

evansolomon added a commit that referenced this pull request Apr 15, 2015
@evansolomon evansolomon merged commit af37d93 into slackapi:master Apr 15, 2015
@austinpray
Copy link

@evansolomon can we get a release with this included?

@ddffx
Copy link

ddffx commented May 8, 2015

@evansolomon any plan to put it in a release?

@goofansu
Copy link

👍 Need a release

@goofansu
Copy link

Hubot says Slack client closed, waiting for reconnect but cannot connect. I've waited for hours.

@goofansu
Copy link

It is terminated. I ran it on my Macbook Pro.

[Tue May 19 2015 13:50:12 GMT+0800 (CST)] INFO Connecting...
[Tue May 19 2015 13:50:32 GMT+0800 (CST)] INFO Logged in as yybot of 上海页游网络科技, but not yet connected
[Tue May 19 2015 13:50:38 GMT+0800 (CST)] INFO Slack client now connected
[Tue May 19 2015 13:50:39 GMT+0800 (CST)] INFO Using default redis on localhost:6379
[Tue May 19 2015 13:50:39 GMT+0800 (CST)] INFO Data for hubot brain retrieved from Redis
[Tue May 19 2015 13:52:43 GMT+0800 (CST)] ERROR Last pong is too old: 14.638
[Tue May 19 2015 13:52:43 GMT+0800 (CST)] INFO Reconnecting in 1000ms
[Tue May 19 2015 13:52:44 GMT+0800 (CST)] INFO Attempting reconnect
[Tue May 19 2015 13:52:44 GMT+0800 (CST)] INFO Connecting...
[Tue May 19 2015 13:53:03 GMT+0800 (CST)] INFO Slack client closed, waiting for reconnect
[Tue May 19 2015 13:53:05 GMT+0800 (CST)] INFO Logged in as yybot of 上海页游网络科技, but not yet connected
[Tue May 19 2015 13:53:07 GMT+0800 (CST)] INFO Slack client now connected
[Tue May 19 2015 13:53:07 GMT+0800 (CST)] INFO Using default redis on localhost:6379
[Tue May 19 2015 13:53:07 GMT+0800 (CST)] INFO Data for hubot brain retrieved from Redis
[Tue May 19 2015 14:00:17 GMT+0800 (CST)] ERROR Last pong is too old: 14.705
[Tue May 19 2015 14:00:17 GMT+0800 (CST)] INFO Reconnecting in 1000ms
[Tue May 19 2015 14:00:18 GMT+0800 (CST)] INFO Attempting reconnect
[Tue May 19 2015 14:00:18 GMT+0800 (CST)] INFO Connecting...
[Tue May 19 2015 14:00:47 GMT+0800 (CST)] INFO Slack client closed, waiting for reconnect
[Tue May 19 2015 14:02:44 GMT+0800 (CST)] ERROR Received error "ECONNRESET"
[Tue May 19 2015 14:02:44 GMT+0800 (CST)] ERROR undefined
[Tue May 19 2015 14:02:44 GMT+0800 (CST)] ERROR Exiting in 1 second
[Tue May 19 2015 14:02:44 GMT+0800 (CST)] INFO Reconnecting in 2000ms

@danielktaylor
Copy link

Tested this fix and it works, but then the listeners do not reinitialize or something. The bot rejoins the channel but does not respond to any commands.

hkdobrev added a commit to OpenBuildings/hubot that referenced this pull request May 27, 2015
This includes slackapi/hubot-slack#177 which
fixes a reconnecting issue.

It also includes the still open PR
slackapi/hubot-slack#195
which enables real mentioning of people according to Slack API.
@technicalpickles
Copy link
Contributor

cc #228 for tracking getting this released

@kimamula
Copy link

Hubot says Slack client closed, waiting for reconnect but cannot connect. I've waited for hours.

I ran into the same problem.
It is likely we have to wait until slackapi/node-slack-sdk#5 to be solved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SlackBot should reconnect instead of terminating the process
8 participants