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

Maximum number of clients reached #3

Closed
lucas91andrade opened this issue Dec 7, 2017 · 2 comments · Fixed by #4
Closed

Maximum number of clients reached #3

lucas91andrade opened this issue Dec 7, 2017 · 2 comments · Fixed by #4

Comments

@lucas91andrade
Copy link

lucas91andrade commented Dec 7, 2017

Hello bithavoc, thanks for the nice pkg, was exactly what I was looking for.
But trying to run this on linux for some days now:

var desktopIdle = require('desktop-idle')
setInterval(function() {
  console.log(desktopIdle.getIdleTime())
}, 1000)

Keeps (after some seconds, failing into this):

┏ Electron -------------------

  1
  
┗ ----------------------------

┏ Electron -------------------

  Maximum number of clients reached

┗ ----------------------------
....

Keeps repeating "1", fills ram and crash PC... HAVOC ! ! heh.
Tried running it from node and from window, same behavior.

Any help much appreciated, thank you!

@bithavoc
Copy link
Owner

bithavoc commented Dec 7, 2017

I may have a Memory leak in the number of X clients to I'm creating for the Linux path. Test suite needs to be imported for this.

bithavoc added a commit that referenced this issue Dec 7, 2017
bithavoc added a commit that referenced this issue Dec 7, 2017
@bithavoc
Copy link
Owner

bithavoc commented Dec 7, 2017

@lucas91andrade please give desktop-idle@1.1.1 a try, it includes the fix for the memory leak.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants