flowdock / hubot-flowdock

Flowdock adapter for Hubot chat bot
94 stars 60 forks source link

Hubot stops sending messages with 0.7.5 #76

Closed Mumakil closed 9 years ago

Mumakil commented 9 years ago

Hubot might stop sending messages when updated to hubot-flowdock 0.7.5 with some configurations. Reverting to 0.7.4 fixes the issue, but an npm update also does this. We need to investigate if there should be a minimum npm requirement for the package.

Probably related to #74.

ChrisLundquist commented 9 years ago

Yep. Version 0.7.5

[Mon Apr 13 2015 23:18:50 GMT+0000 (UTC)] DEBUG Received message { event: 'message',
  tags: [ ':unread:146905' ],
  uuid: 'c7I9-gMbH0DiXrM_',
  to: '146905',
  id: 8876758364,
  content: 'hubot: help',
  sent: 1428967130099,
  app: 'chat',
  attachments: [],
  user: '150031' }

version 0.7.4:

[Mon Apr 13 2015 23:27:47 GMT+0000 (UTC)] DEBUG Received message { event: 'message',
  tags: [ ':unread:146905' ],
  uuid: 'iMoxDA2mSWhBeeAg',
  to: '146905',
  id: 8876761789,
  content: 'hubot: help',
  sent: 1428967667201,
  app: 'chat',
  attachments: [],
  user: '150031' }
[Mon Apr 13 2015 23:27:47 GMT+0000 (UTC)] DEBUG Message 'hubot: help' matched regex //^\s*[@]?hubot[:,]?\s*(?:help\s*(.*)?$)/i/

Looks like the matching logic isn't firing. The first thing I'd check for is a key change.