Skip to content

Don't accept 0 as meaning unlimited anymore#1413

Open
mbettini-ox wants to merge 1 commit intodovecot:mainfrom
mbettini-ox:DOV-8431
Open

Don't accept 0 as meaning unlimited anymore#1413
mbettini-ox wants to merge 1 commit intodovecot:mainfrom
mbettini-ox:DOV-8431

Conversation

@mbettini-ox
Copy link
Contributor

No description provided.

@mbettini-ox mbettini-ox force-pushed the DOV-8431 branch 7 times, most recently from 44a2e9e to 61139d5 Compare February 2, 2026 15:55
multiplied by [[setting,quota_message_percentage]] to get the final value
(in this mailbox or namespace). This is reported as the MESSAGE limit in IMAP
GETQUOTA commands. Using \`0\` as the value means the same as \`unlimited\`.`
GETQUOTA commands.`
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't remove 0 explanation

[[setting,quota_storage_extra]] to get the final value (in this mailbox or
namespace). This is reported as the STORAGE limit in IMAP GETQUOTA commands.
Using \`0\` as the value means the same as \`unlimited\`.`
namespace). This is reported as the STORAGE limit in IMAP GETQUOTA commands.`
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't remove 0

| [[setting,mail_vsize_bg_after_count]] | | 0 is not allowed anymore to mean unlimited<br>use `unlimited` instead |
| [[setting,passdb_default_password_scheme]] | `PLAIN` | `CRYPT` |
| [[setting,quota_message_count]] | | 0 is not allowed anymore to mean unlimited<br>use `unlimited` instead |
| [[setting,quota_storage_size]] | | 0 is not allowed anymore to mean unlimited<br>use `unlimited` instead |
Copy link
Contributor

Choose a reason for hiding this comment

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

remove these two

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.

3 participants