js-promise-socket icon indicating copy to clipboard operation
js-promise-socket copied to clipboard

Possible EventEmitter memory leak detected.

Open QuantBull opened this issue 4 years ago • 1 comments

My project installed [email protected], when I call 'socket.writeAll' in setInterval, some warnings printed on console.

(node:46580) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 error listeners added. Use emitter.setMaxListeners() to increase limit
(node:46580) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added. Use emitter.setMaxListeners() to increase limit
(node:46580) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 finish listeners added. Use emitter.setMaxListeners() to increase limit
(node:46580) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 drain listeners added. Use emitter.setMaxListeners() to increase limit

And I can't listen timeout event with on or once of promise-socket's instance, so I use promiseSocketInstance.socket.once instead. Thanks.

QuantBull avatar Feb 23 '21 02:02 QuantBull

I have the same issue. I'm going to use emitter.setMaxListeners() to 20 and see if it improves things.

      // init socket
      await this._socket.connect(4070, apAddress);
      this._socket.socket.setMaxListeners(20);

clouedoc avatar Dec 05 '21 14:12 clouedoc

Fixed in https://github.com/dex4er/js-promise-socket/releases/tag/v8.0.0

dex4er avatar Jun 21 '24 13:06 dex4er