-
-
Notifications
You must be signed in to change notification settings - Fork 122
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
Change minimum new mail check interval #1678
Comments
See #1477 |
I disagree that impact is huge. The time passes, server power is growing. Most mail clients don't have this restriction, and not all of them implement permanent IMAP connection. And what impact do you mean: to IMAP server or to Http server? Btw, UI setting for check interval is not good now. The user can input any value to this field ant it will be saved. Only tooltip reminds allowed interval of values. I think some kind if verification is required. |
It does say min="5" max="60"
|
Yes, it does say this in tooltip. But the user can ignore it and enter 1 or 120 for example. And this value will be saved. Upper limit (what for is it?) nowhere applied, but there is another part of code which restricts interval to be not shorter than 5 minutes. But the user will see in settings entered value even if it shorter then 5 minutes and will expect that this interval applied. |
Snappymail doesn't allow to automatically check new mail more often than once per 5 minutes. I see it is too restrictive and suggest to decrease it to 1 minute. Yes, it may lead to extra load to mail server, we may warn user in documentation, but let user decide is it acceptable or not.
I prepared a patch which implements this change.
snappymail_checkmailinterval.patch
The text was updated successfully, but these errors were encountered: