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

bot.sleep() can make memory leak #3349

Open
1 task done
sefirosweb opened this issue Apr 7, 2024 · 0 comments
Open
1 task done

bot.sleep() can make memory leak #3349

sefirosweb opened this issue Apr 7, 2024 · 0 comments
Labels
possible bug Stage1 just created by someone new to the project, we don't know yet if it deserves an implementation / a f waiting info from op

Comments

@sefirosweb
Copy link
Contributor

  • The FAQ doesn't contain a resolution to my issue

##Versions

  • mineflayer: any

Detailed description of a problem

I found a bug on bot.sleep, if the bot cant execute sleep they can make a memory leak, the "event" is never closed:

bot.once('sleep', () => {
clearTimeout(timeoutForSleep)
resolve()
})

I saw my bot is increasing because the bot can't sleep

These are all events using my bot at real time, I saw never decrease "sleep" event until bot can sleep

I think they have a simply solution, i will made PR

@sefirosweb sefirosweb added possible bug Stage1 just created by someone new to the project, we don't know yet if it deserves an implementation / a f labels Apr 7, 2024
@sefirosweb sefirosweb changed the title bot,sleep can make memory leak bot.sleep() can make memory leak Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
possible bug Stage1 just created by someone new to the project, we don't know yet if it deserves an implementation / a f waiting info from op
Projects
None yet
Development

No branches or pull requests

2 participants