Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Increase number of logged in IMAP sessions to 50000 #335

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

link2xt
Copy link
Contributor

@link2xt link2xt commented Jul 1, 2024

No description provided.

@link2xt link2xt force-pushed the link2xt/imap-process-limit branch from bce619a to 8534b80 Compare July 1, 2024 16:46
Copy link
Contributor

@hpk42 hpk42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks for investigating and fixing.

@link2xt
Copy link
Contributor Author

link2xt commented Jul 1, 2024

Maybe we should increase it more, if we hit 1024 already than it will be hit again when there are 20 times more users. And 1024 connections is not that many users, many have multi-device, multiple accounts etc.

@hpk42
Copy link
Contributor

hpk42 commented Jul 1, 2024

yes, let's maybe go for 40K (but also dovecot has a client-limit of 20K, which needs to get increased i think)

@link2xt link2xt changed the title Increase number of logged in IMAP sessions to 20000 Increase number of logged in IMAP sessions to 50000 Jul 1, 2024
@link2xt link2xt force-pushed the link2xt/imap-process-limit branch from 8534b80 to f478623 Compare July 1, 2024 17:20
@link2xt
Copy link
Contributor Author

link2xt commented Jul 1, 2024

client_limit is about not logged in users, once they are logged in they don't count. imap process has client limit of 1 as described in the comment.

@link2xt link2xt merged commit 17389b8 into main Jul 1, 2024
4 of 5 checks passed
@link2xt link2xt deleted the link2xt/imap-process-limit branch July 1, 2024 17:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants