NLC implementation is affecting existing auth and audit middleware listeners

[aeweidne]

With the current NLC implementation, the middleware listeners added by auth and audit are being called for every incoming message, which is more often than before. A secondary problem is that the data passed on the additional events is undefined.

The NLC module implements a catchAll listener (nlc.catchall.js) and a receiveListener (nlc.receive.js).

We need to investigate the best way for these 3 modules to work together.

@ngawor FYI

[aeweidne]

I doubt catchAll and listener middleware can play nicely together. The "match on everything" approach of catch all doesn't match up well with listener middleware's run-on-match approach.

[ngawor]

Opened an issue against hubot: hubotio/hubot#1223