enb / enb

Tool for building web projects, BEM bundler.
https://bem.info/toolbox/enb/
Other
134 stars 54 forks source link

warning: possible EventEmitter memory leak detected #129

Open arikon opened 10 years ago

arikon commented 10 years ago

При (не самом) интенсивном использовании enb server через http в консоль периодически валяться ошибки вида:

(node) warning: possible EventEmitter memory leak detected. 11 listeners added. Use emitter.setMaxListeners() to increase limit.
Trace
    at Socket.EventEmitter.addListener (events.js:160:15)
    at Socket.Readable.on (_stream_readable.js:689:33)
    at module.exports (/Users/arikon/projects/bem/islands-components/node_modules/enb/node_modules/send/node_modules/finished/index.js:13:10)
    at SendStream.stream (/Users/arikon/projects/bem/islands-components/node_modules/enb/node_modules/send/lib/send.js:526:3)
    at SendStream.send (/Users/arikon/projects/bem/islands-components/node_modules/enb/node_modules/send/lib/send.js:470:8)
    at /Users/arikon/projects/bem/islands-components/node_modules/enb/node_modules/send/lib/send.js:382:10
    at Object.oncomplete (fs.js:107:15)

Шаги для воспроизведения можно найти по ссылке https://nda.ya.ru/3QTRbx

mdevils commented 10 years ago

Спасибо за сообщение об ошибке. Я приму Pull Request, исправляющий эту ошибку.

arikon commented 10 years ago

@mdevils Мне непонятно, в чём здесь ошибка: действительно ли здесь leak, или достаточно будет через setMaxListeners() лимит поднять.

mdevils commented 10 years ago

Leak'а нет. Нужно поднять лимит.