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

Error in log pointing to possible memory leak #5369

Closed
3 of 11 tasks
johanschelin opened this issue Jan 29, 2023 · 5 comments · Fixed by #5371
Closed
3 of 11 tasks

Error in log pointing to possible memory leak #5369

johanschelin opened this issue Jan 29, 2023 · 5 comments · Fixed by #5371
Projects

Comments

@johanschelin
Copy link

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?

NO, my problem is NOT within Home Assistant or the ZWave JS integration

Is your problem within Z-Wave JS UI (formerly ZwaveJS2MQTT)?

NO, my problem is NOT within Z-Wave JS UI

Checklist

Describe the bug

I am not sure if this is important, but will log the issue to see, as I can't find any useful information from googling this:

What causes the bug?
Not sure

What do you observe?
Only that this error is written in the log.

Steps to reproduce the behavior:
As soon as I do a restart av the Add-on this always arises.

The error that is logged is the following:
(node:157) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 error listeners added to [Keyv]. Use emitter.setMaxListeners() to increase limit
(Use node --trace-warnings ... to show where the warning was created)

Is this something known that I should only disregard, or is it something you think we should dig deeper into?

Device information

Manufacturer:
Model name:
Node ID in your network:

How are you using node-zwave-js?

  • zwave-js-ui (formerly zwavejs2mqtt) Docker image (latest)
  • zwave-js-ui (formerly zwavejs2mqtt) Docker image (dev)
  • zwave-js-ui (formerly zwavejs2mqtt) Docker manually built (please specify branches)
  • ioBroker.zwave2 adapter (please specify version)
  • HomeAssistant zwave_js integration (please specify version)
  • pkg
  • node-red-contrib-zwave-js (please specify version, double click node to find out)
  • Manually built from GitHub (please specify branch)
  • Other (please describe)

Which branches or versions?

zwave-js-ui: 8.8.0
zwave-js: 10.4.0
home id: 4110551221
home hex: 0xf50208b5

Did you change anything?

no

If yes, what did you change?

No response

Did this work before?

Don't know, this is a new device

If yes, where did it work?

No response

Attach Driver Logfile

Will attach log if you think this issue is worth digging into. (Am in the middle of a long running firmware update, so dont want to restart and change loglevel now)

@zwave-js-bot
Copy link
Collaborator

👋 Hey @johanschelin!

Thanks for opening an issue! It doesn't look like you provided a logfile though. While not strictly necessary for every issue, having a driver log is required to diagnose most issues.

Please consider uploading a logfile that captures your problem.

@zwave-js-bot zwave-js-bot added this to Needs triage in Triage Jan 29, 2023
@AlCalzone
Copy link
Member

@robertsLando does Keyv ring a bell with you?

@robertsLando
Copy link
Member

robertsLando commented Jan 30, 2023

@AlCalzone By checking yarn.lock it's a dependency of cacheable-request and esm2cjs modules, the first is also a dep of got that I think is what you are using to make some api calls, could be that maybe?

Yeah I think it's got: sindresorhus/got#1523

@AlCalzone
Copy link
Member

meh...

@AlCalzone
Copy link
Member

AlCalzone commented Jan 30, 2023

That's what you get for not implementing everything yourself 🙄

Triage automation moved this from Needs triage to Closed Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Triage
Closed
Development

Successfully merging a pull request may close this issue.

4 participants