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

[BUG]: the block command stopped working #3131

Closed
lorenzo132 opened this issue Dec 21, 2021 · 5 comments
Closed

[BUG]: the block command stopped working #3131

lorenzo132 opened this issue Dec 21, 2021 · 5 comments
Labels
maybe: bug An unconfirmed bug staged Staged for next version

Comments

@lorenzo132
Copy link
Member

lorenzo132 commented Dec 21, 2021

Bot Version

4.0.0.dev6

How are you hosting Modmail?

Systemd

Error Logs

https://hastebin.cc/omopiqobip

Screenshots

No response

Additional Information

When having to much blocker users the blocked cmd to show the blocked users will stop responding and error out.

@fourjr
Copy link
Collaborator

fourjr commented Apr 19, 2022

Fix attempted in v4-dev8

@fourjr fourjr added the staged Staged for next version label Apr 19, 2022
fourjr added a commit that referenced this issue Apr 19, 2022
TJZine added a commit to TJZine/modmail that referenced this issue May 1, 2022
TJZine added a commit to TJZine/modmail that referenced this issue May 2, 2022
@fourjr fourjr closed this as completed Sep 6, 2022
@lorenzo132
Copy link
Member Author

Reopening this, the list still fails to show when there's 1k+ users blocked.
Quering through the db takes more then 30minutes which makes it impossible to use the command.
This is with both Selfhosted DB's and mongo hosted DB's
cc @StephenDaDev

@lorenzo132
Copy link
Member Author

Reopening this, the list still fails to show when there's 1k+ users blocked. Quering through the db takes more then 30minutes which makes it impossible to use the command. This is with both Selfhosted DB's and mongo hosted DB's cc @StephenDaDev

Coolguy suggested making a cache using redis ( https://discord.com/channels/515071617815019520/528927720063631360/1065390687270473958 ) which would be nice, however i'm afraid the regular discord user does not know how to work with redis

@khakers
Copy link
Contributor

khakers commented Jan 19, 2023

From a quick look at the code, it doesn't seem like the database is involved at all aside from when the bot pulls the config at launch

@khakers
Copy link
Contributor

khakers commented Jan 19, 2023

Should also add, this issue (#3131) seemed to have been timezone related (and not size) so you might want to make a new one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maybe: bug An unconfirmed bug staged Staged for next version
Projects
None yet
Development

No branches or pull requests

3 participants