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

Infinite loop possible under various outcomes #84

Open
iminlikewithyou opened this issue Aug 2, 2023 · 0 comments
Open

Infinite loop possible under various outcomes #84

iminlikewithyou opened this issue Aug 2, 2023 · 0 comments
Labels
priority:1 Important. Feels bad to not address. type:reliability

Comments

@iminlikewithyou
Copy link
Member

https://github.com/omg-games/lame-bots/blob/70ecc3dce8c79298036b8cd37383f1264aa450ae/lame-bot/src/client.js#L52-L70

this code is so extremely janky but I'm not completely sure how to address it
should probably look more into documentation

a theoretical way this could trigger is if a TextChannel would be deleted before the request is made or if the channel was just null in the first place, or some other possible error occurs which just happens repeatedly

in any of those cases it would just make requests infinitely. enough of those and discord will boot that bot offline

@iminlikewithyou iminlikewithyou added priority:1 Important. Feels bad to not address. type:reliability and removed bug labels Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:1 Important. Feels bad to not address. type:reliability
Projects
None yet
Development

No branches or pull requests

1 participant