search

sematext / logsene-cli

Logsene Command-line Interface
http://sematext.com/logsene
Apache License 2.0
10 stars 5 forks source link

Maximum call stack size exceeded on node.js 6.x #3

Closed megastef closed 8 years ago

megastef commented 8 years ago

I think elasticsearch client dependencies need an update: 


logsene search 
events.js:436
EventEmitter.listenerCount = function(emitter, type) {
                                     ^

RangeError: Maximum call stack size exceeded
    at Function.EventEmitter.listenerCount (events.js:436:38)
    at Log.listenerCount (/usr/local/lib/node_modules/logsene-cli/node_modules/elasticsearch/src/lib/log.js:68:25)
    at Function.EventEmitter.listenerCount (events.js:438:20)
    at Log.listenerCount (/usr/local/lib/node_modules/logsene-cli/node_modules/elasticsearch/src/lib/log.js:68:25)
    at Function.EventEmitter.listenerCount (events.js:438:20)
    at Log.listenerCount (/usr/local/lib/node_modules/logsene-cli/node_modules/elasticsearch/src/lib/log.js:68:25)
    at Function.EventEmitter.listenerCount (events.js:438:20)
    at Log.listenerCount (/usr/local/lib/node_modules/logsene-cli/node_modules/elasticsearch/src/lib/log.js:68:25)
    at Function.EventEmitter.listenerCount (events.js:438:20)
    at Log.listenerCount (/usr/local/lib/node_modules/logsene-cli/node_modules/elasticsearch/src/lib/log.js:68:25)
    at Function.EventEmitter.listenerCount (events.js:438:20)
    at Log.listenerCount (/usr/local/lib/node_modules/logsene-cli/node_modules/elasticsearch/src/lib/log.js:68:25)
    at Function.EventEmitter.listenerCount (events.js:438:20)
    at Log.listenerCount (/usr/local/lib/node_modules/logsene-cli/node_modules/elasticsearch/src/lib/log.js:68:25)
    at Function.EventEmitter.listenerCount (events.js:438:20)
    at Log.listenerCount (/usr/local/lib/node_modules/logsene-cli/node_modules/elasticsearch/src/lib/log.js:68:25)
mbonaci commented 8 years ago

Thanks @megastef 👍 , I'll take a look.

megastef commented 8 years ago

I could not reproduce the problem on another machine with the forked repository. But installing from npm caused the error above. So npm publish should help.

megastef commented 8 years ago

Fixed with new npm release logsene-cli@1.4.1 https://www.npmjs.com/package/logsene-cli

mbonaci commented 8 years ago

Thanks