-
Notifications
You must be signed in to change notification settings - Fork 30.2k
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
12.9.0 MaxListenersExceededWarning #29239
Comments
Do you have a minimal code example to reproduce this? |
Sorry, I don't :/ I just noticed this with a simple npm install. |
Simple to reproduce:
|
I can reproduce. Here's the stack trace:
|
Hi @targos, I've caught same warning on Linux. |
I suspect I know what it is. My computer broke so I need to sort that out. Will have a PR later today. |
@targos let’s do a revert with fast-track and get 12.9.1 out of the door asap :/. |
This reverts commit bdf07f4. The commit causes somewhat rampant MaxListenerExceeded warnings on fairly generic npm installs. Fixes: nodejs#29239
If I had a Windows binary I could :) |
Reproduced on OSX as well. |
Same problem:
|
I get : (node:5052) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 drain listeners added to [TLSSocket]. I use windows 7 and it's really frustrating. I get this error when I run |
i got the same error message, with the following code: 12936, while running Ubuntu 18 |
(node:25644) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 drain listeners added to [TLSSocket]. Use emitter.setMaxListeners() to increase limit
Done ? |
Same on MacOS installing (updating) @angular/cli
|
So there is a very recent bug or something? |
@Josufst: It was introduced in 12.9. |
I'm hitting this, reproducibly too - on OSX, Ubuntu, and Ubuntu in Docker :) |
I'm going to re-open this Note that this bug just means you're getting a bunch of warnings, but Not sure yet when someone on @nodejs/releasers will have time to put together a 12.9.1 release, but we have a reasonably brisk release cadence for 12.x so it should be soon. |
Unlocked but please read this mini-FAQ before posting. What version/OS/etc. does this affect? Node.js 12.9.0, all environments Windows, macOS, Linux, Docker, yarn, npm, you-name-it, if it's using Node.js 12.9.0, it's affected. What are the ramifications of this bug? If you're hitting it through I am hitting this same bug on this or that platform using this or that tool. Thanks. Please don't post "this is affecting me too" comments and please don't be offended if your comment gets hidden as "off topic" to try to keep this issue as usable as possible. As noted above, this affects everything on Node.js 12.9.0. When is a fix coming out?
I'm hitting this bug in some other version than Node.js 12.9.0. Probably not, but you may have a different issue triggering the same warning. Open a separate issue in https://github.com/nodejs/help or (if you are reasonably certain the issue is with Node.js itself) in this repository. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
12.9.1 release which fixes this is being prepared and should be out in the next 36 hours or so (although that's merely a prediction based on passed experience, not a promise/guarantee). Reproducing the FAQ above for visibility: What version/OS/etc. does this affect? Node.js 12.9.0, all environments Windows, macOS, Linux, Docker, yarn, npm, you-name-it, if it's using Node.js 12.9.0, it's affected. What are the ramifications of this bug? If you're hitting it through I am hitting this same bug on this or that platform using this or that tool. Thanks. Please don't post "this is affecting me too" comments and please don't be offended if your comment gets hidden as "off topic" to try to keep this issue as usable as possible. As noted above, this affects everything on Node.js 12.9.0. When is a fix coming out? Probably some time on August 27 (UTC). I'm hitting this bug in some other version than Node.js 12.9.0. Probably not, but you may have a different issue triggering the same warning. Open a separate issue in https://github.com/nodejs/help or (if you are reasonably certain the issue is with Node.js itself) in this repository. |
v12.9.1 is out, thanks for the fix! |
Hey guys I just wanna confirm that I had the same bug a couple of hours ago with a message like this
I was on I have updated my nodejs to 🥇 |
This comment has been minimized.
This comment has been minimized.
@SVendittelli: If you are getting this on v10 please open a separate issue. This specific issue was cause by a regression introduced in 12.9.0. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
If you are seeing this message and running Node,js 12.9.0, update to the most recent 12.x. If you are seeing this message with some other version of Node.js, it is not this issue. It is most likely that you have a bug in your code (or there's a bug in one of your dependencies). Or there's no bug but your code is creating a lot of listeners. Or maybe it's a different bug in Node.js. Open an issue in https://github.com/nodejs/help. (Or if you are very certain the bug is in Node.js itself, open an issue in this issue tracker.) I am going to lock this issue. |
Started to happen after updating to 12.9.0. I'm getting:
even when doing
npm i
in a project:Going back to 12.8.1 and the warnings are gone.
The text was updated successfully, but these errors were encountered: