Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(node) warning: possible EventEmitter memory leak detected. 11 listeners added. Use emitter.setMaxListeners() to increase limit #341

Open
janl opened this issue Feb 27, 2015 · 4 comments

Comments

@janl
Copy link
Member

janl commented Feb 27, 2015

via hoodiehq/discussion#72

tl;dr: harmless for now, but should be fixed

@boennemann
Copy link
Member

Related to plugin use, which is not relevant for now.

@bdiz
Copy link

bdiz commented Sep 25, 2017

@janl To get my tests passing I'm having to hack node_modules/@hoodie/store-client/index.js and add the line emitter.setMaxListeners(1000).

I don't like doing this, but I could not figure out a way to get a handle to the emitter from my app.

While this is not yet fixed in hoodie, is there a better way for me to silence these errors in my tests?

@gr2m
Copy link
Member

gr2m commented Sep 25, 2017

@bdiz make sure you have the latest version of everything. If you are still seeing these messages there is likely a memory leak somewhere in https://github.com/hoodiehq/hoodie-store-client, it’d be great if you could devote some time to find out where this is coming from <3

@bdiz
Copy link

bdiz commented Sep 25, 2017

@gr2m I am using hoodie 28.2.2. Looks like this is where it is coming from: https://github.com/hoodiehq/hoodie-store-client/blob/6663e784723735f56cbf46c1d67f5d9c088448a7/lib/helpers/db-put.js#L23

@gr2m gr2m reopened this Oct 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants