Skip to content

Increase number of logged in IMAP sessions to 50000#335

Merged
link2xt merged 1 commit intomainfrom
link2xt/imap-process-limit
Jul 1, 2024
Merged

Increase number of logged in IMAP sessions to 50000#335
link2xt merged 1 commit intomainfrom
link2xt/imap-process-limit

Conversation

@link2xt
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
@link2xt link2xt deleted the link2xt/imap-process-limit branch July 1, 2024 17:20
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 this pull request may close these issues.

2 participants