search

xmppjs / hubot-xmpp

XMPP adapter for Hubot
181 stars 103 forks source link

Check for id in stanza before using it #92

Closed anupdhml closed 8 years ago

anupdhml commented 8 years ago

Should have caught this earlier, but just a missing ?.

Not all stanzas have id set and this was actually causing serious problems: hubot stopped responding and I had to restart it.

anupdhml commented 8 years ago

Was introduced in https://github.com/markstory/hubot-xmpp/pull/90.

The public npm registry doesn't have the latest version so don't think anyone really noticed it. Do you know when the version there will be updated?

markstory commented 8 years ago

I can aim to make a new package tonight.

anupdhml commented 8 years ago

@markstory thanks! I can see it now.