ERROR ReferenceError: room is not defined

Question

ERROR ReferenceError: room is not defined

hhyyrylainen opened this issue 6 years ago · comments

I get this error when the bot tries to post rss entries:

ERROR ReferenceError: room is not defined                     
    at Timeout._onTimeout (/home/hubot/myhubot/node_modules/hubot-rss-rolf/scripts/hubot-rss-reader.coffe
e:71:51)                                                                                                 
    at ontimeout (timers.js:475:11)                                                                      
    at tryOnTimeout (timers.js:310:5)                                                                    
    at Timer.listOnTimeout (timers.js:270:5)

I found a fix for this. Change the setInterval part to look like this:

  setInterval ->
    return if typeof robot.send isnt 'function'
    return if send_queue.length < 1
    msg = send_queue.shift()
    msg.envelope.user = {}
    try
      robot.send msg.envelope, msg.body
    catch err
      logger.error "Error sending message in setInterval"
      logger.error err
  , 2000

Maybe this is caused by using a newer rocketchat connector.

Robert Müller · Answer 1 · Wed Jul 11 2018 21:31:42 GMT+0800 (China Standard Time)

Hi,
does this happen always or only from time to time?

Henri Hyyryläinen · Answer 2 · Wed Jul 11 2018 22:02:48 GMT+0800 (China Standard Time)

Before I made that change I didn't manage to get the bot to post any rss items at all and after that it just worked the next time I registered a new rss feed. So at least for me it happened 100% of the time.

Robert Müller · Answer 3 · Wed Jul 11 2018 22:04:41 GMT+0800 (China Standard Time)

What version of RocketChat are you running?

At least for us the bot still works fine 🤔

Henri Hyyryläinen · Answer 4 · Wed Jul 11 2018 22:05:33 GMT+0800 (China Standard Time)

I used the latest release and the latest hubot rocketchat connector (which might be what has this change)

Robert Müller · Answer 5 · Wed Jul 11 2018 22:10:24 GMT+0800 (China Standard Time)

Unfortunately I am no longer maintaining our RocketChat, if you could paste me some version numbers I'll check back with our instance admin so we can figure out what might have introduced this.

Henri Hyyryläinen · Answer 6 · Wed Jul 11 2018 22:21:29 GMT+0800 (China Standard Time)

My rocket chat version is 0.66.1 and my hubot package versions are (pretty confusingly): hubot-rocketchat@2.0.0 and hubot@2.19.0 but under hubot-rocketchat npm list also prints hubot@3.0.1.

Robert Müller · Answer 7 · Thu Jul 12 2018 15:42:52 GMT+0800 (China Standard Time)

As I already assumed we are running a older version of hubot. It seems that there were some bigger changes which require a lot more work that I thought.
According to this changes the bot should also be rewritten in js :/

https://www.npmjs.com/package/hubot-rocketchat#important

Robert Müller · Answer 8 · Thu Aug 30 2018 17:02:52 GMT+0800 (China Standard Time)

I am going to rewrite the whole bot into ES6 and make in compatible with the new hubot-rocketchat v2 adapter and Hubot v3. I try to keep as much features as possible.

Robert Müller · Answer 9 · Fri Aug 31 2018 20:16:04 GMT+0800 (China Standard Time)

@hhyyrylainen Can you please check if hubot-rss-rolf@2.0.0-beta1 does work somewhat for you?

I've got it running using the following dependencies:

  "dependencies": {
    "hubot": "3",
    "hubot-help": "^1.0.1",
    "hubot-http-status": "^0.2.2",
    "hubot-redis-brain": "^1.0.0",
    "hubot-rocketchat": "github:rocketchat/hubot-rocketchat",
    "hubot-rss-rolf": "2.0.0-beta1"

Henri Hyyryläinen · Answer 10 · Sat Sep 01 2018 00:33:33 GMT+0800 (China Standard Time)

Unfortunately I decided that rocketchat didn't work well enough so I have already removed it so I can't easily test to see if the new version works.

realrill · Answer 11 · Sun Jan 27 2019 05:37:35 GMT+0800 (China Standard Time)

@Flipez I am able to regenerate the issue with 2.0.0-beta1

Could you please take a look on that?

droid_1_165c287c5d29 | [Sat Jan 26 2019 21:34:58 GMT+0000 (Coordinated Universal Time)] INFO Filters passed, will receive message
droid_1_165c287c5d29 | [Sat Jan 26 2019 21:34:58 GMT+0000 (Coordinated Universal Time)] INFO hubot-rss-reader: add http://feeds.skynews.com/feeds/rss/world.xml
droid_1_165c287c5d29 | [Sat Jan 26 2019 21:34:58 GMT+0000 (Coordinated Universal Time)] INFO hubot-rss-reader: checker start
droid_1_165c287c5d29 | [Sat Jan 26 2019 21:34:58 GMT+0000 (Coordinated Universal Time)] INFO [sendMessage] Calling (async): [{"msg":"registered http://feeds.skynews.com/feeds/rss/world.xml","bot":{"i":"js.SDK"},"rid":"qswndTHNDruLKmaiz"}]
droid_1_165c287c5d29 | [Sat Jan 26 2019 21:34:58 GMT+0000 (Coordinated Universal Time)] INFO [received] Message in room qswndTHNDruLKmaiz
droid_1_165c287c5d29 | [Sat Jan 26 2019 21:34:58 GMT+0000 (Coordinated Universal Time)] INFO [received] Message in room qswndTHNDruLKmaiz
droid_1_165c287c5d29 | [Sat Jan 26 2019 21:34:58 GMT+0000 (Coordinated Universal Time)] INFO [received] Message in room qswndTHNDruLKmaiz
droid_1_165c287c5d29 | [Sat Jan 26 2019 21:34:58 GMT+0000 (Coordinated Universal Time)] INFO hubot-rss-reader: wait 600 seconds
droid_1_165c287c5d29 | [Sat Jan 26 2019 21:34:59 GMT+0000 (Coordinated Universal Time)] ERROR ReferenceError: room is not defined
droid_1_165c287c5d29 |   at Timeout._onTimeout (/home/hubot/node_modules/hubot-rss-rolf/scripts/hubot-rss-reader.js:79:61)
droid_1_165c287c5d29 |   at ontimeout (timers.js:436:11)
droid_1_165c287c5d29 |   at tryOnTimeout (timers.js:300:5)
droid_1_165c287c5d29 |   at listOnTimeout (timers.js:263:5)
droid_1_165c287c5d29 |   at Timer.processTimers (timers.js:223:10)
droid_1_165c287c5d29 |

RocketChat 0.73.1
hubot-rss-rolf 2.0.0-beta1