search

aeirola / slack-irc-client

Use Slack as your IRC client
88 stars 16 forks source link

Client fails to start #2

Closed boucher closed 8 years ago

boucher commented 8 years ago 
info: Starting slack-irc-client
info: Connecting to Slack...
/usr/local/lib/node_modules/slack-irc-client/node_modules/slack-client/lib/clients/rtm/client.js:183
    this.logger('verbose', 'attempting to connect via the RTM API');
         ^

TypeError: this.logger is not a function
    at RTMClient.start (/usr/local/lib/node_modules/slack-irc-client/node_modules/slack-client/lib/clients/rtm/client.js:183:10)
    at Object.initSlackListener [as init] (/usr/local/lib/node_modules/slack-irc-client/lib/slack-listener.js:35:15)
    at start (/usr/local/lib/node_modules/slack-irc-client/bin/server.js:80:26)
    at Object.<anonymous> (/usr/local/lib/node_modules/slack-irc-client/bin/server.js:84:1)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Function.Module.runMain (module.js:457:10)
    at startup (node.js:138:18)

Running on OS X, node v5.1.1.

aeirola commented 8 years ago

Thanks for reporting this. Apparently the slack-client package changed how they call the logger (https://github.com/slackhq/node-slack-client/commit/2576ee87c8860acc4141622d730ba1bc2df1553a) in version 2.0.3, breaking backwards compatibility.

I'll update the logger code soon, and make a new release to fix this.

boucher commented 8 years ago

Great, thanks.

aeirola commented 8 years ago

This has now been fixed in version 0.1.5