Skip to content

📚️ Fix QUOTA documentation, ✅ Test #setquota, ♻️ Add MailboxQuota#quota_root alias#636

Merged
nevans merged 2 commits intomasterfrom
document-incomplete-quota-support-and-update-names
Apr 4, 2026
Merged

📚️ Fix QUOTA documentation, ✅ Test #setquota, ♻️ Add MailboxQuota#quota_root alias#636
nevans merged 2 commits intomasterfrom
document-incomplete-quota-support-and-update-names

Conversation

@nevans
Copy link
Copy Markdown
Collaborator

@nevans nevans commented Apr 4, 2026

This updates the QUOTA docs with references to RFC9208 (QUOTA=RES-*). For the most part, RFC9208 is backward compatible with RFC2087. So this updates the documentation to reference it wherever relevant.

This also documents how net-imap's support for QUOTA is incomplete: that it only supports setting STORAGE and can only parse a single resource type (which is usually STORAGE).

The RFCs are very clear that quota root is not the same as mailbox, so this updates method parameters and rdoc to reflect that. MailboxQuota#mailbox is incorrectly named, but renaming it would be backward incompatible, so this just updates the documentation and adds quota_root as an alias.

This also fixes the rdoc for MailboxQuota#quota to specify that it is the storage limit, rather than any other quota resource type. This adds a note in the rdoc that only STORAGE is currently supported.

GETQUOTA generally is available to normal users. That comment on #getquota may have been copied from SETQUOTA.

See also: #622

@nevans nevans force-pushed the document-incomplete-quota-support-and-update-names branch from 2e0b2e9 to a1d90f9 Compare April 4, 2026 13:42
This updates the QUOTA docs with references to RFC9208 (`QUOTA=RES-*`).
For the most part, RFC9208 is backward compatible with RFC2087.  So this
updates the documentation to reference it wherever relevant.

This also documents how `net-imap`'s support for `QUOTA` is incomplete:
that it only supports setting `STORAGE` and can only parse a single
resource type (which is _usually_ `STORAGE`).

The RFCs are very clear that `quota root` is _not_ the same as
`mailbox`, so this updates method parameters and rdoc to reflect that.
`MailboxQuota#mailbox` is incorrectly named, but renaming it would be
backward incompatible.  This just updates the documentation and adds
`quota_root` as an alias.

This also fixes the rdoc for `MailboxQuota#quota` to specify that it is
the _storage_ limit, rather than any other quota resource type.  This
adds a note in the rdoc that _only_ `STORAGE` is currently supported.

`GETQUOTA` generally _is_ available to normal users.  That comment on
`#getquota` may have been copied from `SETQUOTA`.

See also: #622
@nevans nevans force-pushed the document-incomplete-quota-support-and-update-names branch from a1d90f9 to 0ea729c Compare April 4, 2026 13:45
@nevans nevans merged commit 888ba70 into master Apr 4, 2026
39 checks passed
@nevans nevans deleted the document-incomplete-quota-support-and-update-names branch April 4, 2026 13:49
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.

1 participant